-
Notifications
You must be signed in to change notification settings - Fork 7.2k
Remove custom ops interpolation with antialiasing #5329
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
Conversation
💊 CI failures summary and remediationsAs of commit 1e8caca (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).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
@vfdev-5 The failures seem related. |
Oh, I was thinking that pytorch/pytorch#71937 is already in nightly ... Let's wait then with this PR. |
Summary: Description: - Added missing antialias argument to functional.pyi.in - mypy is happy if checking `interpolate` method with antialias argument Related torchvision issue: pytorch/vision#5329 Pull Request resolved: #72420 Reviewed By: mruberry Differential Revision: D34043182 Pulled By: albanD fbshipit-source-id: ca64a8f0d2516c1be5b060c1c24e0b1ed2127b96
Summary: Description: - Added missing antialias argument to functional.pyi.in - mypy is happy if checking `interpolate` method with antialias argument Related torchvision issue: pytorch/vision#5329 Pull Request resolved: #72420 Reviewed By: mruberry Differential Revision: D34043182 Pulled By: albanD fbshipit-source-id: ca64a8f0d2516c1be5b060c1c24e0b1ed2127b96 (cherry picked from commit 7c8a90c)
Summary: Description: - Added missing antialias argument to functional.pyi.in - mypy is happy if checking `interpolate` method with antialias argument Related torchvision issue: pytorch/vision#5329 Pull Request resolved: pytorch/pytorch#72420 Reviewed By: mruberry Differential Revision: D34043182 Pulled By: albanD fbshipit-source-id: ca64a8f0d2516c1be5b060c1c24e0b1ed2127b96 (cherry picked from commit 7c8a90cbfa54b4369644bf69cce1eb162f66b3f7)
Summary: Description: - Added missing antialias argument to functional.pyi.in - mypy is happy if checking `interpolate` method with antialias argument Related torchvision issue: pytorch/vision#5329 Pull Request resolved: pytorch/pytorch#72420 Reviewed By: mruberry Differential Revision: D34043182 Pulled By: albanD fbshipit-source-id: ca64a8f0d2516c1be5b060c1c24e0b1ed2127b96 (cherry picked from commit 7c8a90cbfa54b4369644bf69cce1eb162f66b3f7)
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.
LGTM, thanks!
Summary: * Removed custom ops for interp with AA * Fixed umft issues Reviewed By: sallysyw Differential Revision: D34265746 fbshipit-source-id: f30177bf2fd625b8b2b923e4db45180779a2e6ad Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
Description:
PyTorch now suppports interpolation op with antialiasing: https://pytorch.org/docs/master/generated/torch.nn.functional.interpolate.html#torch.nn.functional.interpolate
In this PR I remove prelimiary and temporary private custom ops.