Skip to content

Upgrade Transformers to minimum v4.50.0 everywhere #2003

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.

Sign up for GitHub

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

Jump to bottom
Closed
ashahba wants to merge 10 commits into from
Closed

Upgrade Transformers to minimum v4.50.0 everywhere #2003

ashahba wants to merge 10 commits into from

Conversation

@ashahba
Copy link
Collaborator

@ashahba ashahba commented May 27, 2025

Description

HF Transformers olders than v4.50.0 need to be upgraded and that's what this PR is for.

Issues

N/A

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

No new dependencies

Tests

Nothing specific except for the CI runs where Dockerfile or scripts have been touched.

Copilot AI review requested due to automatic review settings May 27, 2025 18:43
Copilot AI reviewed May 27, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • HybridRAG/tests/test_compose_on_gaudi.sh: Language not supported

@github-actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

@rbrugaro rbrugaro mentioned this pull request May 27, 2025
Merged
4 tasks
@ashahba
Copy link
Collaborator Author

ashahba commented May 28, 2025

This should be merged only after opea-project/GenAIComps#1748 is merged.

@joshuayao joshuayao added this to OPEA Jun 4, 2025
@joshuayao joshuayao added this to the v1.4 milestone Jun 4, 2025
@ashahba
Copy link
Collaborator Author

ashahba commented Jun 13, 2025

No longer needed!

@ashahba ashahba closed this Jun 13, 2025
@github-project-automation github-project-automation bot moved this to Done in OPEA Jun 13, 2025
@ashahba ashahba deleted the ashahba/transformers-4.50.0-upgrade branch June 13, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@lvliang-intel lvliang-intel Awaiting requested review from lvliang-intel lvliang-intel is a code owner

@ftian1 ftian1 Awaiting requested review from ftian1 ftian1 is a code owner

@chensuyue chensuyue Awaiting requested review from chensuyue chensuyue is a code owner

@lkk12014402 lkk12014402 Awaiting requested review from lkk12014402 lkk12014402 is a code owner

@minmin-intel minmin-intel Awaiting requested review from minmin-intel minmin-intel is a code owner

@rbrugaro rbrugaro Awaiting requested review from rbrugaro rbrugaro is a code owner

Assignees

No one assigned

Labels

None yet

Projects

OPEA
Status: Done

Milestone

v1.4

Development

Successfully merging this pull request may close these issues.

3 participants

@ashahba @joshuayao @ZePan110