Skip to content

Commit 3456b1a

Browse files
committed
.
1 parent f6e6187 commit 3456b1a

File tree

1 file changed

+3
-31
lines changed

1 file changed

+3
-31
lines changed

.github/workflows/linux_cuda_wheel.yaml

Lines changed: 3 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ jobs:
6363
matrix:
6464
python-version: ['3.9', '3.10', '3.11', '3.12']
6565
cuda-version: ['12.4']
66-
ffmpeg-version-for-tests: ['6']
66+
ffmpeg-version-for-tests: ['5', '6', '7']
6767
container:
6868
image: "pytorch/manylinux-builder:cuda${{ matrix.cuda-version }}"
6969
options: "--gpus all -e NVIDIA_DRIVER_CAPABILITIES=video,compute,utility"
@@ -76,8 +76,8 @@ jobs:
7676
echo cuda_version_without_periods=${cuda_version_without_periods} >> $GITHUB_ENV
7777
- uses: actions/download-artifact@v3
7878
with:
79-
# This was originally:
80-
# name: pytorch_torchcodec__${{ matrix.python-version }}_cu${{ env.cuda_version_without_periods }}_x86_64
79+
# We use 3.9 because it's the minimum version we build.
80+
# We test version 3.9 against all other python versions in the matrix.
8181
name: pytorch_torchcodec__3.9_cu${{ env.cuda_version_without_periods }}_x86_64
8282
path: pytorch/torchcodec/dist/
8383
- name: Setup miniconda using test-infra
@@ -95,7 +95,6 @@ jobs:
9595
- name: Install PyTorch
9696
run: |
9797
${CONDA_RUN} python -m pip install --pre torch --index-url https://download.pytorch.org/whl/nightly/cu${{ env.cuda_version_without_periods }}
98-
# ${CONDA_RUN} python -m pip install torch torchvision torchaudio
9998
${CONDA_RUN} python -c 'import torch; print(f"{torch.__version__}"); print(f"{torch.__file__}"); print(f"{torch.cuda.is_available()=}")'
10099
- name: Install torchcodec from the wheel
101100
run: |
@@ -105,25 +104,6 @@ jobs:
105104
106105
- name: Check out repo
107106
uses: actions/checkout@v3
108-
- name: Build & install ffmpeg after installing torchcodec
109-
run: |
110-
# Ideally we would have checked for that before installing the wheel,
111-
# but we need to checkout the repo to access this file, and we don't
112-
# want to checkout the repo before installing the wheel to avoid any
113-
# side-effect. It's OK.
114-
# source packaging/helpers.sh
115-
# assert_ffmpeg_not_installed
116-
# It seems like conda can't install ffmpeg cleanly in many cases. So we just build it from source.
117-
118-
# ${CONDA_RUN} conda install --yes ffmpeg=${{ matrix.ffmpeg-version-for-tests }} -vv -c conda-forge
119-
# ${CONDA_RUN} conda install --yes conda
120-
# ${CONDA_RUN} conda update --all --yes
121-
# ${CONDA_RUN} conda config --add channels conda-forge
122-
# ${CONDA_RUN} conda config --set channel_priority strict
123-
# ${CONDA_RUN} conda config --show
124-
# ${CONDA_RUN} $CONDA_PREFIX/bin/conda install --yes ffmpeg=${{ matrix.ffmpeg-version-for-tests }} -c conda-forge
125-
# ${CONDA_INSTALL} --yes ffmpeg=${{ matrix.ffmpeg-version-for-tests }} -c conda-forge
126-
# ${CONDA_RUN} conda install --yes ffmpeg=${{ matrix.ffmpeg-version-for-tests }} -c conda-forge
127107

128108
- name: Install cuda runtime dependencies
129109
run: |
@@ -150,14 +130,6 @@ jobs:
150130
ls
151131
- name: Smoke test
152132
run: |
153-
${CONDA_RUN} find $CONDA_PREFIX -name libav\*.so | xargs ldd
154-
${CONDA_RUN} find $CONDA_PREFIX -name libav\*.so | xargs readelf -d
155-
${CONDA_RUN} find / -name libnvcuvid\*
156-
find / -name libnvcuvid\*
157-
${CONDA_RUN} find $CONDA_PREFIX -name libtorchcodec\*.so | xargs ldd
158-
${CONDA_RUN} find $CONDA_PREFIX -name libtorchcodec\*.so | xargs readelf -d
159-
${CONDA_RUN} find $CONDA_PREFIX -name libcuda\*.so\*
160-
${CONDA_RUN} echo $CONDA_PREFIX
161133
${CONDA_RUN} python test/decoders/manual_smoke_test.py
162134
- name: Run Python tests
163135
run: |

0 commit comments

Comments
 (0)