-
Notifications
You must be signed in to change notification settings - Fork 525
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
pt: fix loss training when no data available #3571
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## devel #3571 +/- ##
=======================================
Coverage 77.48% 77.49%
=======================================
Files 432 432
Lines 37182 37202 +20
Branches 1620 1620
=======================================
+ Hits 28812 28831 +19
- Misses 7502 7503 +1
Partials 868 868 ☔ View full report in Codecov by Sentry. |
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.
See @wanghan-iapcm's comment in #3047
Fixed in d5e9f88 |
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.
How do you test if the loss work property when the label is absent?
fix in 602a060 |
Fix #3482 and #3483.