Skip to content

[CI] Install torchcodec from source#3541

Open
vmoens wants to merge 6 commits intomainfrom
local-install-torchcodec2
Open

[CI] Install torchcodec from source#3541
vmoens wants to merge 6 commits intomainfrom
local-install-torchcodec2

Conversation

@vmoens
Copy link
Collaborator

@vmoens vmoens commented Mar 4, 2026

No description provided.

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 4, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/rl/3541

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

❌ 1 New Failure, 2 Cancelled Jobs, 7 Unrelated Failures

As of commit dfd76ce with merge base 491ed0f (image):

NEW FAILURE - The following job has failed:

CANCELLED JOBS - The following jobs were cancelled. Please retry:

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 This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 4, 2026
@github-actions github-actions bot added the CI Has to do with CI setup (e.g. wheels & builds, tests...) label Mar 4, 2026
vmoens added 5 commits March 4, 2026 17:00
torchcodec's setup.py requires this env var when building a wheel
to handle FFmpeg bundling. Without it, the build fails with a
ValueError about license confirmation.

Made-with: Cursor
- Remove torchcodec from main pip install list since it's now handled
  by the dedicated source/release install block below.
- Export BUILD_AGAINST_ALL_FFMPEG_FROM_S3 so the env var propagates
  through the uv_pip_install shell function to the build subprocess.

Made-with: Cursor
CMake's FindPython3 can't locate Python include headers in uv's
standalone Python installation. Setting Python3_ROOT_DIR to the
base prefix lets CMake find the development files.

Made-with: Cursor
- Convert video frames to uint8 in _write_video before passing to
  VideoEncoder, which requires uint8 input.
- Update MLFlow test_log_video to use torchcodec VideoDecoder for
  read-back verification when available.
- Install torchcodec in docs build environment to fix sphinx gallery
  failures when torchvision >= 0.22.

Made-with: Cursor
@github-actions github-actions bot added the Record label Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Has to do with CI setup (e.g. wheels & builds, tests...) CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Record

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant