Skip to content

fix cuda ut fail for transformers v5.0 - #1357

Merged
xin3he merged 20 commits into
mainfrom
hengguo/fix_v5_cuda_ut
Jan 30, 2026
Merged

fix cuda ut fail for transformers v5.0#1357
xin3he merged 20 commits into
mainfrom
hengguo/fix_v5_cuda_ut

Conversation

@n1ck-guo

@n1ck-guo n1ck-guo commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

Description

Please briefly describe your main changes, the motivation.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Other (please specify):

Related Issues

Fixes or relates to #1315

Checklist Before Submitting

  • My code has been tested locally.
  • Documentation has been updated as needed.
  • New or updated tests are included where applicable.

Signed-off-by: n1ck-guo <heng.guo@intel.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request addresses test failures in CUDA unit tests caused by breaking changes in transformers v5.0. The changes add conditional test skipping using version checks to maintain test compatibility across different transformers versions.

Changes:

  • Added version checking and conditional test skipping for transformers v5.0
  • Updated imports to include transformers_version helper and packaging.version
  • Skipped tests affected by breaking changes in MOE models, GGUF format support, MT5Tokenizer removal, and other transformers v5.0 incompatibilities

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test/test_cuda/schemes/test_scheme.py Added skip condition for MOE model test affected by transformers v5.0 breaking changes
test/test_cuda/models/test_support_vlms.py Added skip decorator to test_glm for transformers <4.54.0 version constraint
test/test_cuda/models/test_get_block_name.py Added skip condition for test_flux due to MT5Tokenizer removal in transformers v5.0
test/test_cuda/models/test_diffusion.py Added class-level skip decorator for TestAutoRound due to MT5Tokenizer removal
test/test_cuda/export/test_gguf.py Added class-level skip decorator for GGUF format issues in transformers v5.0
test/test_cuda/advanced/test_fp8_input.py Added skip conditions for multiple tests affected by GGUF and other transformers v5.0 changes

Comment thread test/test_cuda/export/test_gguf.py
Comment thread test/test_cuda/advanced/test_fp8_input.py
Comment thread test/test_cuda/advanced/test_fp8_input.py
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
@xin3he
xin3he marked this pull request as draft January 28, 2026 06:12
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Comment thread test/test_cuda/models/test_moe_model.py Outdated
Comment thread auto_round/eval/eval_cli.py Outdated
…checks in tests

Signed-off-by: He, Xin3 <xin3.he@intel.com>
@xin3he
xin3he marked this pull request as ready for review January 28, 2026 08:36
Comment thread test/test_cuda/advanced/test_fp8_input.py
Comment thread test/test_cuda/advanced/test_fp8_input.py Outdated
@xin3he
xin3he requested a review from wenhuach21 January 28, 2026 09:06
xin3he and others added 10 commits January 29, 2026 10:54
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
@xin3he
xin3he merged commit eb2ccdc into main Jan 30, 2026
28 checks passed
@xin3he
xin3he deleted the hengguo/fix_v5_cuda_ut branch January 30, 2026 02:00
lvliang-intel pushed a commit that referenced this pull request Feb 2, 2026
Signed-off-by: n1ck-guo <heng.guo@intel.com>
Signed-off-by: He, Xin3 <xin3.he@intel.com>
Co-authored-by: He, Xin3 <xin3.he@intel.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@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.

4 participants