-
Notifications
You must be signed in to change notification settings - Fork 22.1k
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
[numpy] torch.i0: promote integer inputs to float #52735
[numpy] torch.i0: promote integer inputs to float #52735
Conversation
💊 CI failures summary and remediationsAs of commit 7eb9b74 (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 to the (internal) Dr. CI Users group. |
@@ -1972,6 +1972,24 @@ def wrapped_fn(x): | |||
return wrapped_fn | |||
|
|||
|
|||
def np_unary_ufunc_integer_promotion_wrapper_with_astype(fn): |
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.
Copied from #56352
Codecov Report
@@ Coverage Diff @@
## master #52735 +/- ##
==========================================
- Coverage 76.80% 76.80% -0.01%
==========================================
Files 1986 1986
Lines 198237 198247 +10
==========================================
+ Hits 152255 152259 +4
- Misses 45982 45988 +6 |
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.
Cool! I'm still not in love with that astype function but we don't need to sort out anything better at the moment. It's good enough.
Nice work, @kshitij12345
@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
I know :). Will clean it up in #56352 (assuming this gets merged first) Thanks :) |
Summary: Reference : pytorch#42515 Pull Request resolved: pytorch#52735 Reviewed By: zou3519 Differential Revision: D28630505 Pulled By: mruberry fbshipit-source-id: e81a35dfc1a322daf0c44718901470fac677bc94
Reference : #42515