Skip to content

Deprecate AutoModelForVision2Seq - #38900

Merged
zucchini-nlp merged 6 commits into
huggingface:mainfrom
zucchini-nlp:deprecate-vision2seq
Jul 14, 2025
Merged

zucchini-nlp merged 6 commits into
huggingface:mainfrom
zucchini-nlp:deprecate-vision2seq

Conversation

@zucchini-nlp

Copy link
Copy Markdown
Member

What does this PR do?

As per title, we'll remove it anyway in the end so let's start raising warnings. I already ask all new models to not use Vision2Seq

@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
zucchini-nlp requested a review from Cyrilvallez July 1, 2025 12:54
@Cyrilvallez

Copy link
Copy Markdown
Member

I don't remember all the discussions around this, but is ImageTextToText what will be fully future-proof? So we don't go with ForCausalLM in the end?

@zucchini-nlp

Copy link
Copy Markdown
Member Author

I don't think we'll remove ImageTextToText in the near future though there were discussions on unifying "AutoForMultimodal" under one umbrella mapping. didn't yet start working on new auto class, but I think it will be a new auto-mapping that:

  1. Unifies all generative model, either under one class or two classes (one for text and other for multimodality)
  2. Main goal is to clean up garbage repetitive code and we don't plan to delete existing auto classes, otherwise it might break a lot of external libraries that depend on us. Until new auto is added, we'll have a surge of new VLMs added to the mapping which makes it harder to just remove it

Personally, I'm in favor of keeping ImageTextToText for a while as the recommended mapping

@Cyrilvallez Cyrilvallez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@github-actions

Copy link
Copy Markdown
Contributor

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

run-slow: auto

@zucchini-nlp
zucchini-nlp merged commit 878d60a into huggingface:main Jul 14, 2025
rjgleaton pushed a commit to rjgleaton/transformers that referenced this pull request Jul 17, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
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