Skip to content

Bump transformers upper bound to include 5.3.0#502

Merged
YujiaBao merged 2 commits intothinking-machines-lab:mainfrom
YujiaBao:bump-transformers-upper-bound
Mar 20, 2026
Merged

Bump transformers upper bound to include 5.3.0#502
YujiaBao merged 2 commits intothinking-machines-lab:mainfrom
YujiaBao:bump-transformers-upper-bound

Conversation

@YujiaBao
Copy link
Copy Markdown
Member

@YujiaBao YujiaBao commented Mar 20, 2026

Summary

  • Update dependency constraint from <5.3.0 to <=5.3.0 in pyproject.toml
  • Bump CI test matrix (pytest + pyright) from 5.2.0 to 5.3.0
  • Skip DeepSeek tokenizer tests on exactly transformers==5.3.0 due to an upstream bug that strips spaces during decode (huggingface/transformers#44801). Tests will resume on 5.3.1+.
  • Add a runtime warning when DeepSeek renderers are initialized with transformers==5.3.0

Test plan

  • CI passes with transformers 5.3.0 (DeepSeek tests skipped)
  • CI passes with transformers 4.57.6 (all tests run normally)
  • Runtime warning is emitted when using DeepSeek renderer with transformers 5.3.0

🤖 Generated with Claude Code

Update dependency constraint from <5.3.0 to <=5.3.0 and bump CI test
matrix from 5.2.0 to 5.3.0.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@YujiaBao YujiaBao requested a review from dphuang2 March 20, 2026 00:59
transformers 5.3.0 has a known bug that strips spaces during decode
for the DeepSeek tokenizer (huggingface/transformers#44801). Skip
affected tests on exactly 5.3.0 (will resume on 5.3.1+) and emit a
runtime warning when DeepSeek renderers are initialized with 5.3.0.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@YujiaBao YujiaBao requested a review from aurickq March 20, 2026 01:24
@YujiaBao YujiaBao merged commit 0be951b into thinking-machines-lab:main Mar 20, 2026
6 checks passed
@YujiaBao YujiaBao deleted the bump-transformers-upper-bound branch March 27, 2026 16:26
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