-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[Bugfix] Fix Apertus HF repo name #24447
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
There was a problem hiding this 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 correctly updates the Hugging Face repository name for the Apertus model in the test suite and model registry. It also adds the model to the supported models documentation. I have one suggestion to improve the consistency of the documentation.
Emu3 test is failing now: https://buildkite.com/vllm/ci/builds/29803/steps/canvas?jid=01992aa7-239e-4392-9ae0-d5ed04ea3c2d |
I see, let me look into this (seems it's broken at core code about configuration)... 😅 |
huggingface/transformers#40766 should fix this. |
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk> Signed-off-by: xuebwang-amd <xuebwang@amd.com>
Purpose
The HF repo got renamed since the initial release #23068. Also add this model to Supported Models page.
Fix Transformers Nightly Models Test https://buildkite.com/vllm/ci/builds/29705/steps/canvas?sid=01992255-a7b7-4ec3-bfde-a3cc263a7e3c
Test Plan
Test Result
Essential Elements of an Effective PR Description Checklist
supported_models.md
andexamples
for a new model.