-
Notifications
You must be signed in to change notification settings - Fork 25k
changed launch bounds, unrolled for loop for grid sampler 2d fwd and bwd #60405
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 e0aa52e (more details on the Dr. CI page and at hud.pytorch.org/pr/60405):
1 failure not recognized by patterns:
❄️ 2 failures tentatively classified as flakybut reruns have not yet been triggered to confirm:
|
@ngimel has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: This PR fixes the `mypy` failure introduced by [`numpy` 1.21.0](https://github.com/numpy/numpy/releases/tag/v1.21.0) (by pinning `numpy` to 1.20, at least for now) and the `quick-checks` failure introduced by #60405. Pull Request resolved: #60472 Test Plan: The Lint workflow in GitHub Actions. Reviewed By: walterddr Differential Revision: D29313009 Pulled By: driazati fbshipit-source-id: 53fd0e0549c26be5fc5d3c502c5891c56c83a32c
Summary: This PR fixes the `mypy` failure introduced by [`numpy` 1.21.0](https://github.com/numpy/numpy/releases/tag/v1.21.0) (by pinning `numpy` to 1.20, at least for now) and the `quick-checks` failure introduced by pytorch#60405. Pull Request resolved: pytorch#60472 Test Plan: The Lint workflow in GitHub Actions. Reviewed By: walterddr Differential Revision: D29313009 Pulled By: driazati fbshipit-source-id: 53fd0e0549c26be5fc5d3c502c5891c56c83a32c
Changed launch bounds for grid sampler 2d fwd and bwd from 1024 to 256, added loop unrolling to fix register spilling into local memory.
Timing Data: (using Nvidia Titan-V)
Interpolation mode 2, padding 0, align corners False