Skip to content

Conversation

@DN6
Copy link
Collaborator

@DN6 DN6 commented Nov 17, 2025

What does this PR do?

Temporarily pin transformers (and by extension hf_hub) to version 4.57.1 and 0.36. This is to reduce noise in our CI while transformers is under going refactors for 5.0.0. We can unpin after the next Diffusers release.

Fixes # (issue)

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@DN6 DN6 requested a review from sayakpaul November 17, 2025 16:26
Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

uv pip uninstall transformers huggingface_hub && uv pip install --prerelease allow -U transformers@git+https://github.com/huggingface/transformers.git

This is still installing from main no?

uv pip install -e ".[quality]"
uv pip uninstall accelerate && uv pip install -U accelerate@git+https://github.com/huggingface/accelerate.git
#uv pip uninstall transformers huggingface_hub && uv pip install --prerelease allow -U transformers@git+https://github.com/huggingface/transformers.git
uv pip uninstall transformers huggingface_hub && uv pip install transformers
Copy link
Member

Choose a reason for hiding this comment

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

But we don't need this step no? The Docker container should already have a stable version of transformers installed? 👀

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

I would say we keep it uniform and do uv pip uninstall transformers huggingface_hub && uv pip install transformers==4.57.1 everywhere applicable?

@sayakpaul
Copy link
Member

Failing tests seem unrelated.

@sayakpaul sayakpaul merged commit b7df4a5 into main Nov 18, 2025
22 of 24 checks passed
@sayakpaul sayakpaul deleted the ci-pin branch November 18, 2025 09:13
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