Skip to content

Remove all output_attentions and the like#47

Open
Sebmono wants to merge 11 commits intomainfrom
update_all_decorators-migrated-pr-43590
Open

Remove all output_attentions and the like#47
Sebmono wants to merge 11 commits intomainfrom
update_all_decorators-migrated-pr-43590

Conversation

@Sebmono
Copy link

@Sebmono Sebmono commented Feb 1, 2026

Original PR: huggingface/transformers#43590
Opened by: +molbap
Created at: 2026-01-29 12:23:56 UTC
Base branch: mainHead branch: update_all_decorators


What does this PR do?

In model additions, we often see old standards not using check_model_inputs, can_return_tuple and it's often a first review comment/something that can slip through. Doing a wide scan to try to remove all occurrences systematically.

@Sebmono
Copy link
Author

Sebmono commented Feb 1, 2026

Comment by +HuggingFaceDocBuilderDev (2026-01-29 12:33:14 UTC):

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.

@Sebmono
Copy link
Author

Sebmono commented Feb 1, 2026

Comment by +github-actions[bot] (2026-01-30 17:27:38 UTC):

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

run-slow: aimv2, altclip, clap, clip, clipseg, git, groupvit, idefics, kosmos2, metaclip_2, owlv2, owlvit, phi4_multimodal, siglip, siglip2, vipllava

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