Skip to content

Reductions on losses that have dynamic size #523

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.

Sign up for GitHub

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

Jump to bottom
Closed
nfeybesse opened this issue Feb 27, 2024 · 0 comments
Closed

Reductions on losses that have dynamic size #523

nfeybesse opened this issue Feb 27, 2024 · 0 comments

Comments

@nfeybesse
Copy link
Contributor

Currently, AUTO and SUM_OVER_BATCH reduction operations on all losses fail when the batch size is dynamically defined. These reduction operations are very frequently used. Dynamic definition of batch size as well.

The problem comes from the LossesHelper class which seeks to divide the sum of the error by the size of the batch which is not dynamically obtained currently
nfeybesse added a commit to nfeybesse/tensorflow that referenced this issue Feb 27, 2024
@nfeybesse
Fix tensorflow#523
17951df
@nfeybesse nfeybesse changed the title Reductions on size losses that have dynamic size Reductions on losses that have dynamic size Feb 27, 2024
Craigacp pushed a commit that referenced this issue Mar 8, 2024
@nfeybesse
fix #526 (#527)
2b6d83f 
* Interface should be public for external usage

* Fix #523

* Fix google format

* fix #526

* Add test to CategoricalCrossentropyTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nfeybesse