Skip to content

🚨 [Ernie 4.5 VL Moe] Fix up namings to vllm/sglang convention#44299

Merged
vasqu merged 19 commits intohuggingface:mainfrom
vasqu:rename-ernie-vl
Feb 26, 2026
Merged

🚨 [Ernie 4.5 VL Moe] Fix up namings to vllm/sglang convention#44299
vasqu merged 19 commits intohuggingface:mainfrom
vasqu:rename-ernie-vl

Conversation

@vasqu
Copy link
Copy Markdown
Contributor

@vasqu vasqu commented Feb 26, 2026

As per title, to allow for proper alignment with vllm/sglang

Closes #44258

@vasqu vasqu marked this pull request as draft February 26, 2026 12:51
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@vasqu
Copy link
Copy Markdown
Contributor Author

vasqu commented Feb 26, 2026

run-slow: ernie4_5_vl_moe

@github-actions
Copy link
Copy Markdown
Contributor

Workflow Run ⚙️

This comment contains run-slow, running the specified jobs:

models: ["models/ernie4_5_vl_moe"]
quantizations: []

@github-actions
Copy link
Copy Markdown
Contributor

CI Results

Workflow Run ⚙️

Commit Info

Context Commit Description
RUN 06f3acaf workflow commit (merge commit)
PR 80d628b6 branch commit (from PR)
main d4cb8416 base commit (on main)

⚠️ No test being reported (jobs are skipped or cancelled)!

@vasqu
Copy link
Copy Markdown
Contributor Author

vasqu commented Feb 26, 2026

run-slow: ernie4_5_vl_moe

@github-actions
Copy link
Copy Markdown
Contributor

Workflow Run ⚙️

This comment contains run-slow, running the specified jobs:

models: ["models/ernie4_5_vl_moe"]
quantizations: []

model="baidu/ERNIE-4.5-VL-28B-A3B-PT",
device_map="auto",
revision="refs/pr/10",
revision="refs/pr/11",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

New revision because the old one had a merge conflict and I had no idea how to resolve it, mirrored it basically

```

## Ernie4_5_VL_MoeConfig
## Ernie4_5_VLMoeConfig
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

VL_Moe <-> VLMoe to match sglang and vllm

Comment on lines +395 to +396
# NOTE: Slightly different from `model_type` (to follow naming conventions in vllm/sglang)
"ernie4_5_vlmoe",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Weird case where we check for the model class name so it's not the same as model type, just as a note no to be confused

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These are dependent on the hub so CI might be red if people are currently working

@github-actions
Copy link
Copy Markdown
Contributor

CI Results

Workflow Run ⚙️

Commit Info

Context Commit Description
RUN e689abd4 workflow commit (merge commit)
PR 6ae9ec7e branch commit (from PR)
main d4cb8416 base commit (on main)

✅ No failing test specific to this PR 🎉 👏 !

@vasqu vasqu marked this pull request as ready for review February 26, 2026 14:36
@github-actions
Copy link
Copy Markdown
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: auto, ernie4_5_vl_moe

@vasqu
Copy link
Copy Markdown
Contributor Author

vasqu commented Feb 26, 2026

Force merging this since it is blocking CI; run slow passed everything and BC alias runs as well

@vasqu vasqu merged commit b812aa9 into huggingface:main Feb 26, 2026
25 checks passed
@vasqu vasqu deleted the rename-ernie-vl branch February 26, 2026 16:42
zvik pushed a commit to zvik/transformers that referenced this pull request Mar 1, 2026
…ingface#44299)

* rename

* small fix and and adjust converter

* oops

* try

* docs

* fix tests

* last one?

* revisions

* fixup naming for ckpt conversion needed for norm

* trigger ci fresh

* aliases with classes because modular doesnt pick it up otherwise

* style

* so many exceptions

* fix

* when does it end

* small fixup

* add bc name to conversion mapping as well
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