Skip to content

Update docker image to use av==10.0.0#38548

Merged
ydshieh merged 2 commits into
mainfrom
fix_av_docker_image
Jun 3, 2025
Merged

Update docker image to use av==10.0.0#38548
ydshieh merged 2 commits into
mainfrom
fix_av_docker_image

Conversation

@ydshieh

@ydshieh ydshieh commented Jun 3, 2025

Copy link
Copy Markdown
Collaborator

What does this PR do?

Docker image is failing to build, as 9.2.0 wheel is removed, see

PyAV-Org/PyAV#1906 (comment)

Failing job: https://github.com/huggingface/transformers/actions/runs/15079790825/job/42394494171


# For video model testing
RUN python3 -m pip install --no-cache-dir av==9.2.0
RUN python3 -m pip install --no-cache-dir av==10.0.0

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

since it's video, so cc @zucchini-nlp or @qubvel ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Or if you prefer, we can remove the version pin.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yeah, I think we can remove the pin, we don't require specific version for video utilities

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done

@ydshieh ydshieh changed the title Update Update docker image to use av==10.0.0 Jun 3, 2025
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@ydshieh ydshieh requested a review from zucchini-nlp June 3, 2025 08:55

@zucchini-nlp zucchini-nlp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

❤️

@ydshieh ydshieh merged commit ca0a682 into main Jun 3, 2025
11 checks passed
@ydshieh ydshieh deleted the fix_av_docker_image branch June 3, 2025 09:04
bvantuan pushed a commit to bvantuan/transformers that referenced this pull request Jun 12, 2025
* Update

* Update

---------

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants