-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Actually run backward criterion tests. #44030
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
This looks to have been a mistake from #9287. [ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit d50cc53 (more details on the Dr. CI page):
ci.pytorch.org: 1 failedThis comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 8 times. |
This looks to have been a mistake from #9287. Differential Revision: [D23476274](https://our.internmc.facebook.com/intern/diff/D23476274) [ghstack-poisoned]
|
I turned off the CTCLoss tests for now because they fail. |
Codecov Report
@@ Coverage Diff @@
## gh/gchanan/314/base #44030 +/- ##
======================================================
Coverage ? 69.42%
======================================================
Files ? 381
Lines ? 47168
Branches ? 0
======================================================
Hits ? 32747
Misses ? 14421
Partials ? 0 Continue to review full report at Codecov.
|
zou3519
left a comment
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.
I can't believe we didn't notice this for two years... Thanks for the fix!
Stack from ghstack:
This looks to have been a mistake from #9287.
Differential Revision: D23476274