Skip to content

[v5] remove load_in_4bit and load_in_8bit#41287

Merged
SunMarc merged 9 commits into
mainfrom
rm-bnb-load-in
Oct 9, 2025
Merged

[v5] remove load_in_4bit and load_in_8bit#41287
SunMarc merged 9 commits into
mainfrom
rm-bnb-load-in

Conversation

@SunMarc

@SunMarc SunMarc commented Oct 2, 2025

Copy link
Copy Markdown
Member

What does this PR do?

This PR removes load_in_4bit and load_in_8bit deprecated arguments for v5. This is something that I didn't want to do in a minor release as this might be breaking for a lot of users.

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

@ydshieh

ydshieh commented Oct 2, 2025

Copy link
Copy Markdown
Collaborator

Could you run a few integration tests to make sure 🙏 Thank you.

@MekkCyber MekkCyber left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Finallyyyy 🔥 ! Great clean up

@ydshieh ydshieh added the for_v5? label Oct 2, 2025
Comment thread src/transformers/quantizers/auto.py
@SunMarc SunMarc requested a review from ArthurZucker October 2, 2025 14:09
@SunMarc SunMarc mentioned this pull request Oct 9, 2025
@github-actions

github-actions Bot commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

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

run-slow: aya_vision, chameleon, colqwen2, diffllama, emu3, ernie4_5_moe, esm, falcon, gemma

@SunMarc

SunMarc commented Oct 9, 2025

Copy link
Copy Markdown
Member Author

run-slow: bnb

@SunMarc

SunMarc commented Oct 9, 2025

Copy link
Copy Markdown
Member Author

run-slow: gemma

@github-actions

github-actions Bot commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

This comment contains run-slow, running the specified jobs:

models: ['models/gemma']
quantizations: [] ...

@SunMarc SunMarc merged commit b44d915 into main Oct 9, 2025
26 of 27 checks passed
@SunMarc SunMarc deleted the rm-bnb-load-in branch October 9, 2025 14:34
@ArthurZucker

ArthurZucker commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator

hahha have it in #41445 as well for from_pretrained

AhnJoonSung pushed a commit to AhnJoonSung/transformers that referenced this pull request Oct 12, 2025
* [v5] remove load_in_4bit and load_in_8bit

* fix

* reveert

* fix

---------

Co-authored-by: Mohamed Mekkouri <93391238+MekkCyber@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants