Skip to content

Overwrite get_decoder() in AudioLLMs#42896

Merged
zucchini-nlp merged 2 commits intohuggingface:mainfrom
zucchini-nlp:voxtral-get-decoder
Dec 17, 2025
Merged

Overwrite get_decoder() in AudioLLMs#42896
zucchini-nlp merged 2 commits intohuggingface:mainfrom
zucchini-nlp:voxtral-get-decoder

Conversation

@zucchini-nlp
Copy link
Copy Markdown
Member

What does this PR do?

As per title, audio models need to call LM's get_decoder() because the model has no base model

@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.

@zucchini-nlp
Copy link
Copy Markdown
Member Author

@bot /style

@github-actions
Copy link
Copy Markdown
Contributor

Style fix is beginning .... View the workflow run here.

Copy link
Copy Markdown
Contributor

@molbap molbap left a comment

Choose a reason for hiding this comment

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

Sounds good - these are the only two in this case?

@zucchini-nlp
Copy link
Copy Markdown
Member Author

Yes, also there is Qwen2-Audio but it already has the method overriden

@zucchini-nlp
Copy link
Copy Markdown
Member Author

@bot /style

@zucchini-nlp zucchini-nlp enabled auto-merge (squash) December 17, 2025 10:32
@github-actions
Copy link
Copy Markdown
Contributor

Style fix is beginning .... View the workflow run here.

@github-actions
Copy link
Copy Markdown
Contributor

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

run-slow: granite_speech

@zucchini-nlp
Copy link
Copy Markdown
Member Author

Voxtral had it overriden already 🤦🏻 I misread the GH comment, so it's only GraniteSpeech

@zucchini-nlp zucchini-nlp merged commit 1dc69bd into huggingface:main Dec 17, 2025
19 checks passed
SangbumChoi pushed a commit to SangbumChoi/transformers that referenced this pull request Jan 23, 2026
audio models don't define LM as base model, overwrite explicitly!
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.

3 participants