Skip to content
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

fix error test for F.pad_mask #8132

Merged
merged 2 commits into from
Nov 23, 2023
Merged

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented Nov 23, 2023

Spotted in #8127 / https://github.com/pytorch/vision/actions/runs/6971354622/attempts/1#summary-18971247416, because it failed to torch.compile test.

Since we only want to trigger the error and we have checked the kernel above already, we can just call it directly here.

cc @vfdev-5

Copy link

pytorch-bot bot commented Nov 23, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/8132

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (33 Unrelated Failures)

As of commit 15f4216 with merge base 3eb4333 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if green, thanks!

@pmeier pmeier merged commit ec7f80b into pytorch:main Nov 23, 2023
1 check passed
@pmeier pmeier deleted the fix-pad-mask-test-error branch November 23, 2023 18:59
facebook-github-bot pushed a commit that referenced this pull request Jan 16, 2024
Reviewed By: vmoens

Differential Revision: D52539021

fbshipit-source-id: 9467e50fa68ee76ec55c16244e5eb4f88c33a795
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants