Skip to content

Use timm-side buffers initialization#43124

Merged
Cyrilvallez merged 9 commits intomainfrom
timm-buffers
Jan 7, 2026
Merged

Use timm-side buffers initialization#43124
Cyrilvallez merged 9 commits intomainfrom
timm-buffers

Conversation

@Cyrilvallez
Copy link
Copy Markdown
Member

What does this PR do?

As per the title. Clean-up timm buffers initialization that was temporarily added in #42941. It has now been upstreamed directly in timm, so let's simplify on the Transformers side!

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

@rwightman
Copy link
Copy Markdown
Contributor

rwightman commented Jan 6, 2026

@Cyrilvallez shouldn't this use init_non_persistent_buffers() instead of _init_buffers()?

A related FYI, there was a missed import error (in a deprecated path) due to a moved module location, still have to support those as lots of code uses it so will be doing a 1.0.24 release very shortly to fix it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 6, 2026

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

run-slow: gemma3n, timm_backbone, timm_wrapper

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 6, 2026

View the CircleCI Test Summary for this PR:

https://huggingface.co/spaces/transformers-community/circle-ci-viz?pr=43124&sha=cbd492

@Cyrilvallez Cyrilvallez merged commit 8dd9c99 into main Jan 7, 2026
24 of 26 checks passed
@Cyrilvallez Cyrilvallez deleted the timm-buffers branch January 7, 2026 12:59
SangbumChoi pushed a commit to SangbumChoi/transformers that referenced this pull request Jan 23, 2026
* make the switch

* pin version

* don't protect import if it is used without protection....

* add batchnorm2d

* fix

* fix

* oupsi

* use public API

* fix typo
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.

4 participants