Skip to content
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.{rad2deg, deg2rad}: promote integer inputs to float #51853

Conversation

kshitij12345
Copy link
Collaborator

Reference: #42515

Depends on #51283

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Feb 7, 2021

💊 CI failures summary and remediations

As of commit 1b2f573 (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.

@codecov
Copy link

codecov bot commented Feb 7, 2021

Codecov Report

Merging #51853 (1b2f573) into master (ee04cd9) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #51853      +/-   ##
==========================================
- Coverage   80.77%   80.76%   -0.01%     
==========================================
  Files        1969     1969              
  Lines      216063   216076      +13     
==========================================
+ Hits       174515   174517       +2     
- Misses      41548    41559      +11     

@kshitij12345 kshitij12345 marked this pull request as ready for review February 11, 2021 03:52
Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mruberry
Copy link
Collaborator

Like #51956, would you rebase this and I'll work on getting it landed? Sorry about the break due to PTO.

@kshitij12345
Copy link
Collaborator Author

No worries. Thanks for looking into it. :)

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@mruberry merged this pull request in a06cf5d.

aocsa pushed a commit to Quansight/pytorch that referenced this pull request Mar 15, 2021
…torch#51853)

Summary:
Reference: pytorch#42515

Depends on pytorch#51283

Pull Request resolved: pytorch#51853

Reviewed By: albanD

Differential Revision: D26399743

Pulled By: mruberry

fbshipit-source-id: a6f0e12723e1451c6479d818752fe5d41788715d
xsacha pushed a commit to xsacha/pytorch that referenced this pull request Mar 31, 2021
…torch#51853)

Summary:
Reference: pytorch#42515

Depends on pytorch#51283

Pull Request resolved: pytorch#51853

Reviewed By: albanD

Differential Revision: D26399743

Pulled By: mruberry

fbshipit-source-id: a6f0e12723e1451c6479d818752fe5d41788715d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants