Skip to content

Add pytest skip to test_contiguity #1121

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.

Sign up for GitHub

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

Jump to bottom
Merged
Dan-Flores merged 1 commit into from
Dec 10, 2025
Merged

Add pytest skip to test_contiguity #1121

Dan-Flores merged 1 commit into from
Dec 10, 2025

Conversation

@Dan-Flores
Copy link
Contributor

@Dan-Flores Dan-Flores commented Dec 10, 2025

The VideoEncoder::test_contiguity test is flaky, and fails most consistently on FFmpeg4 + CUDA. Lets disable it for now to reduce the noise on CI.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Dec 10, 2025
@Dan-Flores Dan-Flores marked this pull request as ready for review December 10, 2025 13:38
NicolasHug
NicolasHug approved these changes Dec 10, 2025
View reviewed changes
Copy link
Contributor

@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.

Thanks @Dan-Flores

@Dan-Flores Dan-Flores merged commit 5c2106f into meta-pytorch:main Dec 10, 2025
73 checks passed
@Dan-Flores Dan-Flores deleted the skip_contiguity_ffmpeg4_cuda branch December 10, 2025 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NicolasHug NicolasHug NicolasHug approved these changes

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Dan-Flores @NicolasHug