-
Notifications
You must be signed in to change notification settings - Fork 7k
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
Improved functional tensor geom transforms to work on floatX dtype #2661
Improved functional tensor geom transforms to work on floatX dtype #2661
Conversation
- Fixes pytorch#2600 - added tests - refactored test_affine
…-5/issue-2600-support-floatX-types
Codecov Report
@@ Coverage Diff @@
## master #2661 +/- ##
==========================================
- Coverage 72.41% 72.15% -0.27%
==========================================
Files 95 95
Lines 8245 8379 +134
Branches 1308 1365 +57
==========================================
+ Hits 5971 6046 +75
- Misses 1858 1921 +63
+ Partials 416 412 -4
Continue to review full report at Codecov.
|
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.
Just one comment about float16, otherwise looks great!
Let me know if you disagree with the float16 reasoning.
…-2600-support-floatX-types
…Quansight/vision into vfdev-5/issue-2600-support-floatX-types
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.
Thanks a lot @vfdev-5 !
…ytorch#2661) * Improved functional tensor geom transforms to work on floatX dtype - Fixes pytorch#2600 - added tests - refactored test_affine * Removed float16/cpu case
…ytorch#2661) * Improved functional tensor geom transforms to work on floatX dtype - Fixes pytorch#2600 - added tests - refactored test_affine * Removed float16/cpu case
Fixes #2600
Description:
Improved functional tensor geom transforms to work on floatX dtype