Skip to content

Conversation

samanklesaria
Copy link
Collaborator

@samanklesaria samanklesaria commented Sep 10, 2025

This PR is similar to #4098, except that here we handle GPU tests rather than CPU ones.

Copy link

pytorch-bot bot commented Sep 10, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/4103

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

❌ 4 New Failures, 2 Pending, 3 Unrelated Failures

As of commit 5b30c64 with merge base caba63f (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following jobs 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.

@meta-cla meta-cla bot added the CLA Signed label Sep 10, 2025
@samanklesaria samanklesaria changed the title Add torchcodec inspired linux gpu CI workflow Use modern ffmpeg in linux gpu tests Sep 10, 2025
@samanklesaria samanklesaria marked this pull request as ready for review September 10, 2025 17:53
@samanklesaria samanklesaria requested a review from a team as a code owner September 10, 2025 17:53
@pearu pearu added this to the 2.9 milestone Sep 11, 2025
@pearu pearu modified the milestones: 2.9, 2.10 Sep 11, 2025
@pearu pearu changed the title Use modern ffmpeg in linux gpu tests Revise Linux GPU unittests CI workflow and remove ffmpeg CI workflow. Sep 12, 2025
@pearu pearu self-requested a review September 12, 2025 08:59
Copy link
Collaborator

@pearu pearu left a comment

Choose a reason for hiding this comment

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

To avoid OOM, linux.g5.4xlarge.nvidia.gpu is now used.

LGTM! Thanks, @samanklesaria!

@pearu pearu merged commit 799d288 into main Sep 12, 2025
41 of 48 checks passed
@pearu pearu deleted the alt-gpu-test-workflow branch September 12, 2025 12:10
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.

2 participants