Skip to content

Conversation

@NicolasHug
Copy link
Member

PLEASE NOTE THAT THE TORCHAUDIO REPOSITORY IS NO LONGER ACTIVELY MONITORED. You may not get a response. For open discussions, visit https://discuss.pytorch.org/.

@NicolasHug NicolasHug requested a review from a team as a code owner September 11, 2025 12:39
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 11, 2025

🔗 Helpful Links

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

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

❌ 34 New Failures, 1 Cancelled Job

As of commit 494bbb4 with merge base caba63f (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOB - The following job was cancelled. Please retry:

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

@meta-cla meta-cla bot added the CLA Signed label Sep 11, 2025
@NicolasHug
Copy link
Member Author

Something seems wrong with the wheel build jobs as they're installing torch 2.8 instead of 2.9RC

++ export BUILD_VERSION=2.9.0+cpu
++ BUILD_VERSION=2.9.0+cpu
++ export PATH=/opt/python/cp39-cp39/bin:/opt/rh/gcc-toolset-13/root/usr/bin:/opt/conda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
++ PATH=/opt/python/cp39-cp39/bin:/opt/rh/gcc-toolset-13/root/usr/bin:/opt/conda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
++ export 'PIP_INSTALL_TORCH=pip install torch==2.8.0 --index-url https://download.pytorch.org/whl/test/cpu'
++ PIP_INSTALL_TORCH='pip install torch==2.8.0 --index-url https://download.pytorch.org/whl/test/cpu'

@pearu
Copy link
Collaborator

pearu commented Sep 11, 2025

Something seems wrong with the wheel build jobs as they're installing torch 2.8 instead of 2.9RC

See

https://github.com/pytorch/test-infra/blob/e1f9a88d4563a71ddbbf3fc1b6948872a7831e2e/.github/workflows/build_wheels_linux.yml#L184

IIUC, we'll need to add stable_version: "2.9" to the matrix sections of wheel jobs.

PYTORCH_WHEEL_INDEX="https://download.pytorch.org/whl/${UPLOAD_CHANNEL}/${GPU_ARCH_ID}"
pip install --progress-bar=off --pre torch torchcodec --index-url="${PYTORCH_WHEEL_INDEX}"
pip install --progress-bar=off --pre torch --index-url="${PYTORCH_WHEEL_INDEX}"
pip install --progress-bar=off --pre torchcodec --index-url="https://download.pytorch.org/whl/nightly/${GPU_ARCH_ID}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does it mean that torchcodec must be released before torchaudio?

Copy link
Collaborator

Choose a reason for hiding this comment

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

According to https://github.com/pytorch/torchcodec/actions/runs/17490234874/job/49679289728, torchcodec 0.7.0 wheels are built against torch 2.8.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sort of, but to be more complete on what the issue is:

The previous

        pip install --progress-bar=off --pre torch torchcodec --index-url="${PYTORCH_WHEEL_INDEX}"

would be trying to install both torch and torchcodec from the test channel, where the RCs are. Torch publishes RCs in there, but we haven't set that up in TorchCodec yet. So it's getting a very very old torchcodec version that we had pushed to test channel in the past, and it's not compatible with torch.

The workaround you see here just installs torchcodec from the nightly channel instead, hoping that it's ABI and API-compatible with torch. It's hacky a brittle, but it's the best we have until torchcodec pushes its RC wheel in the test channel

@NicolasHug
Copy link
Member Author

Waiting on https://github.com/pytorch/test-infra/pull/7150/files to land which should hopefully fix the torch version mismatch

@NicolasHug NicolasHug merged commit e3c6ee2 into pytorch:release/2.9 Sep 11, 2025
5 of 10 checks passed
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