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
Correctly mark unannotated NamedTuple field to be inferred TensorType #46969
Conversation
💊 CI failures summary and remediationsAs of commit b059224 (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This 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 21 times. |
bad0b00
to
2947832
Compare
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.
Nice work, just a few minor comments
2a2bd4b
to
6ecfa64
Compare
6ecfa64
to
57bf21f
Compare
57bf21f
to
115b9db
Compare
Summary: If there is no annotation given, we want to show users that the type is inferred Test Plan: Added a new test case that throws an error with the expected error message Reviewers: Yanan Cao Subscribers: Tasks: pytorch#46326 Tags:
115b9db
to
b059224
Compare
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.
@gmagogsfm has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Codecov Report
@@ Coverage Diff @@
## master #46969 +/- ##
=======================================
Coverage 68.88% 68.88%
=======================================
Files 435 435
Lines 56309 56309
=======================================
Hits 38790 38790
Misses 17519 17519 |
@gmagogsfm merged this pull request in fee585b. |
Summary: If there is no annotation given, we want to show users that the type is inferred
Test Plan: Added a new test case that throws an error with the expected error message
Reviewers: Yanan Cao
Subscribers:
Tasks:
Tags:
Fixes #46326