-
Notifications
You must be signed in to change notification settings - Fork 215
Generic cleanup Metrics and Losses #204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Sync with master tensorflow on upstream
Merge main branch to local branch
Update after losses merge
Fix Javadoc errors (tensorflow#152)
pull type def
Metrics Phase 1 (tensorflow#180)
…when I build it reverses these 2 from master's version.
…er for predictions instead of <T>.
Hi @JimClarke5 , there are some javadoc issues in this PR, please take a look: https://github.com/tensorflow/java/pull/204/checks?check_run_id=1825661890 |
BTW, do you have any clue why the javadoc has been reformatted? It looks like it was already conform before, I don't know why your IDE plugin would reformat these, can you please just double-check that you use your original settings? |
@karllessard I fixed all the JavaDoc issues under |
...-framework/src/main/java/org/tensorflow/framework/metrics/SparseCategoricalCrossentropy.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @JimClarke5 , that simplifies really a lot some of the APIs. I just left a really-unimportant comment about something that looks like a trailing character, please confirm and/or fix and we are good to merge
Ok nevermind, I've remove the trailing character directly from your branch, this is now ready to merge, thanks! |
This PR Is a reissue of the Generic Cleanup PR #203, removing commits in the branch that are already in
master
.This PR effects both
losses
andmetrics
.This PR cleans up generic parameters to cut down on the number of unique parameters to at most 1 where possible.
For example:
public abstract class Metric<U extends TNumber, T extends TNumber> {
becomes
public abstract class Metric<T extends TNumber> {
and
Changes to: