Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CI and update Transformers (4.51.1) #2225

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Fix CI and update Transformers (4.51.1) #2225

wants to merge 30 commits into from

Conversation

IlyasMoutawwakil
Copy link
Member

@IlyasMoutawwakil IlyasMoutawwakil commented Apr 6, 2025

What does this PR do?

Fixes the many CI issues by:

  • Relaying all/most hub related communication to transformers methods (cached_file) since they're more maintained and can handle CI loads (TRANSFORMERS_IS_CI=true)
  • Needed to update transformers to get @echarlaix's argmax fix in Fix ONNX export for sequence classification head  transformers#36332.
  • Needed latest transformers 4.51.1 to get a fix that was part of the patch (weight initialization).
  • used a really small whisper model (cuts onnxruntime testing ci by 40%)
  • Mock torch_pre_mlir to pass MPT remote code tests.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

Who can review?

@HuggingFaceDocBuilderDev

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.

@IlyasMoutawwakil IlyasMoutawwakil requested a review from xenova April 9, 2025 11:47
@IlyasMoutawwakil IlyasMoutawwakil changed the title Fix CI Fix CI and update Transformers (4.51.1) Apr 9, 2025
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.

2 participants