-
Notifications
You must be signed in to change notification settings - Fork 7.1k
[CI] Restore MacOS test sanity #9120
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
DeformConv is only implemented for forward on MPS right now, so skip backward checks Logic for modifying markers in place are stolen from `torch.testing._internal.optests`, namely from https://github.com/pytorch/pytorch/blob/310e8361c565ca1602e719e4c812dc3931ec84d7/torch/testing/_internal/optests/generate_tests.py#L269
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/9120
Note: Links to docs will display an error until the docs builds have been completed. ⏳ No Failures, 29 PendingAs of commit 3c7a7ad with merge base 89f5855 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
test/test_ops.py
Outdated
@@ -1201,11 +1201,30 @@ def test_forward_scriptability(self): | |||
torch.jit.script(ops.DeformConv2d(in_channels=8, out_channels=8, kernel_size=3)) | |||
|
|||
|
|||
# NS: Removeme once bacward is implemented |
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.
Nit: typos
Hey @malfet! You merged this PR, but no labels were added. |
Summary: DeformConv is only implemented for forward on MPS right now, so skip backward checks Logic for modifying markers in place are stolen from `torch.testing._internal.optests`, namely from https://github.com/pytorch/pytorch/blob/310e8361c565ca1602e719e4c812dc3931ec84d7/torch/testing/_internal/optests/generate_tests.py#L269 Reviewed By: AntoineSimoulin Differential Revision: D79175050 fbshipit-source-id: 8b0ee6bad5adb9cc30ce2deeb8ea659b90d73a52
DeformConv is only implemented for forward on MPS right now, so skip
backward checks
Logic for modifying markers in place are stolen from
torch.testing._internal.optests
, namely from https://github.com/pytorch/pytorch/blob/310e8361c565ca1602e719e4c812dc3931ec84d7/torch/testing/_internal/optests/generate_tests.py#L269