-
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
upsample_bilinear2d
and upsample_bicubic2d
: port to structured
#52012
Conversation
[ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit dfaa5ab (more details on the Dr. CI page):
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. |
…red" Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
ghstack-source-id: 4dc26276076057761fbf3c74f7cb7642fa638259 Pull Request resolved: #52012
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
…red" Going ahead and porting the other upsample ops to structured since they follow a similar pattern. Differential Revision: [D26356329](https://our.internmc.facebook.com/intern/diff/D26356329) [ghstack-poisoned]
Codecov Report
@@ Coverage Diff @@
## gh/bdhirsh/78/base #52012 +/- ##
===================================================
Coverage 80.76% 80.76%
===================================================
Files 1969 1969
Lines 216005 215958 -47
===================================================
- Hits 174454 174417 -37
+ Misses 41551 41541 -10 |
output_height, | ||
output_width); | ||
|
||
output.resize_({nbatch, channels, output_height, output_width}, input.suggest_memory_format()); |
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.
Looks like we don't suggest memory_format
here either, the same as in https://github.com/pytorch/pytorch/pull/52065/files#r589557711
cc @vfdev-5
…h#52012) Summary: Pull Request resolved: pytorch#52012 Test Plan: Imported from OSS Reviewed By: ezyang Differential Revision: D26356329 Pulled By: bdhirsh fbshipit-source-id: 8f974224799493e3172fe5dff3fbd43af8c09722
…h#52012) Summary: Pull Request resolved: pytorch#52012 Test Plan: Imported from OSS Reviewed By: ezyang Differential Revision: D26356329 Pulled By: bdhirsh fbshipit-source-id: 8f974224799493e3172fe5dff3fbd43af8c09722
upsample_bilinear2d
and upsample_bicubic2d
: port to structured (#52012)
upsample_bilinear2d
and upsample_bicubic2d
: port to structured (#52012)upsample_bilinear2d
and upsample_bicubic2d
: port to structured
Going ahead and porting the other upsample ops to structured since they follow a similar pattern.
Stack from ghstack:
upsample_nearest3d
andupsample_trilinear3d
: port to structured #52065 port upsample_nearest3d and upsample_trilinear3d to structuredupsample_bilinear2d
andupsample_bicubic2d
: port to structured #52012 port upsample_bilinear2d and upsample_bicubic2d to structuredupsample_linear1d
: port to structured #51917 port upsample_linear1d to structuredDifferential Revision: D26356329