Skip to content

Conversation

@yuki-97
Copy link
Contributor

@yuki-97 yuki-97 commented Jul 29, 2025

vllm<0.10.0 and transformers>=4.54.0 has a conflict which will cause the error below.
This PR limit transformers version to workaround it.

Error Log:

    raise ValueError(f"'{key}' is already used by a Transformers config, pick another name.")
ValueError: 'aimv2' is already used by a Transformers config, pick another name.

We could remove this limit once we upgrade vllm to >=0.10.0 since vllm-project/vllm#20921 fixed this.

Signed-off-by: Yuki Huang <[email protected]>
@yuki-97 yuki-97 added the CI:L1 Run doctests, unit tests, and functional tests label Jul 29, 2025
Signed-off-by: Yuki Huang <[email protected]>
@yuki-97 yuki-97 force-pushed the yukih/transformers-version branch from 9a64f1a to c9ec58c Compare July 29, 2025 15:12
@terrykong terrykong added this pull request to the merge queue Jul 29, 2025
Merged via the queue into main with commit 1c5c189 Jul 29, 2025
15 checks passed
@terrykong terrykong deleted the yukih/transformers-version branch July 29, 2025 19:25
xxman-google pushed a commit to xxman-google/NeMo-RL that referenced this pull request Jul 30, 2025
FannYYW pushed a commit to xxman-google/NeMo-RL that referenced this pull request Aug 5, 2025
FannYYW pushed a commit to xxman-google/NeMo-RL that referenced this pull request Aug 5, 2025
soodoshll pushed a commit to soodoshll/RL that referenced this pull request Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:L1 Run doctests, unit tests, and functional tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants