Reconsider Loss generic parameter
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- java, tensorflow
- Domaine
- backend-api-design, machine-learning
Piste de recherche
Commencez par la signature de méthode générique de Loss présentée dans l’issue et examinez l’API Loss environnante ainsi que ses appelants. Consultez la discussion existante dans les commentaires pour vérifier l’accord sur les relations entre les types. La tâche est terminée lorsque la conception générique est décidée et que l’API et les appelants concernés reflètent cette décision de manière cohérente.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I brought this up with @JimClarke5 but wanted to get some wider comments on it.
The generic parameter for Losses doesn't seem right:
<T extends TNumber> Operand<T> call(Ops tf, Operand<? extends TNumber> labels, Operand<T> predictions, Operand<T> sampleWeights);
It binds the loss value, the predictions, and the weights to the same data type. There is no relation there: it's easy enough to imagine float-weighted cosine similarity of one hot vector predictions, which would return a float. The only necessary data type parameter I can think of is the return type, and that should be on the class.
- Langage dominant
- Java
- Étoiles
- 928
- Forks
- 227
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de tensorflow/java
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
tensorflow/java#653 · 1 commentaire · 4 réactions ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
tensorflow/java#621 · 4 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
tensorflow/java#617 · 3 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 55/100
tensorflow/java#615 · 1 commentaire ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
tensorflow/java#614 · 1 commentaire ·
Toutes les issues de tensorflow/java
Issues similaires
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Ouvertearea/plugin
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
kestra-io/plugin-kestra#190 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Add canonical URLs and a sitemap Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
apache/rocketmq-dashboard#5064 ·