Skip to content

Fix Gemma3 GGUF for Transformers v5#33683

Merged
hmellor merged 1 commit intovllm-project:mainfrom
hmellor:gemma3-v5
Feb 3, 2026
Merged

Fix Gemma3 GGUF for Transformers v5#33683
hmellor merged 1 commit intovllm-project:mainfrom
hmellor:gemma3-v5

Conversation

@hmellor
Copy link
Member

@hmellor hmellor commented Feb 3, 2026

huggingface/transformers#41314 removed from_text_vision_configs as it's a long deprecated method which serves no purpose (you can just use __init__).

This PR updates the Gemma3Config instantiation for GGUF models to use __init__ so that they may continue to work with Transformers v5.

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the instantiation of Gemma3Config for GGUF models to be compatible with Transformers v5. The change correctly replaces the deprecated from_text_vision_configs method with the standard __init__ constructor. This is a necessary and well-implemented fix to maintain compatibility with the upstream library.

@hmellor hmellor enabled auto-merge (squash) February 3, 2026 10:49
@hmellor hmellor added the ready ONLY add when PR is ready to merge/full CI is needed label Feb 3, 2026
@hmellor hmellor merged commit be8168f into vllm-project:main Feb 3, 2026
44 checks passed
@hmellor hmellor deleted the gemma3-v5 branch February 3, 2026 12:36
Hivenet-Igor pushed a commit to Hivenet-Igor/vllm that referenced this pull request Feb 3, 2026
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Igor Carrara <igor.carrara@hivenet.com>
PiratePai pushed a commit to PiratePai/epd_shm that referenced this pull request Feb 3, 2026
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Pai <416932041@qq.com>
PiratePai pushed a commit to PiratePai/epd_shm that referenced this pull request Feb 3, 2026
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Pai <416932041@qq.com>
gameofdimension pushed a commit to gameofdimension/vllm that referenced this pull request Feb 5, 2026
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: felix01.yu <felix01.yu@vipshop.com>
ItzDEXX pushed a commit to ItzDEXX/vllm that referenced this pull request Feb 19, 2026
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants