Add sequence classification support for small Gemma 3 text models#40562
Add sequence classification support for small Gemma 3 text models#40562zucchini-nlp merged 16 commits intohuggingface:mainfrom
Conversation
zucchini-nlp
left a comment
There was a problem hiding this comment.
Hey @abdokaseb , thanks for the PR! Indeed, models saved without an image tower can't be loaded with existing class, totally forgot about them
|
Failing tests are not related, a rebase might help. I will merge a PR to fix processor tests in an hour or so |
|
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. |
|
@abdokaseb, the workers are crashing and thus we can't merge the PR yet. I will check what is wrong and merge when fixed |
|
[For maintainers] Suggested jobs to run (before merge) run-slow: auto, gemma3 |
|
The CI is very flaky lately 😿 |
@zucchini-nlp @abdokaseb Thank you for carrying it over the finish line nevertheless! |
|
🤗 |
What does this PR do?
Add Gemma3TextForSequenceClassification for Gemma3 small text models following generic practice
Testing:
Fixes #40554
Before submitting
Pull Request section?
to it if that's the case.
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
@ArthurZucker