Skip to content

FIX Bug when dequantizing 4bit bnb weights - #2847

Merged
BenjaminBossan merged 1 commit into
huggingface:mainfrom
BenjaminBossan:fix-dequantizing-4bit-bnb
Oct 27, 2025
Merged

BenjaminBossan merged 1 commit into
huggingface:mainfrom
BenjaminBossan:fix-dequantizing-4bit-bnb

Conversation

@BenjaminBossan

@BenjaminBossan BenjaminBossan commented Oct 17, 2025 •

Copy link
Copy Markdown
Member

Fixes some failing GPU tests in CI.

A bug was introduced in #2797 where state.SCB was accessed while dequantizing 4bit bnb weights even though state is None. This would occur, for instance, when using DoRA, which needs to dequantize the weight. The attribute access is now restricted to 8bit bnb weights.

Fixes some failing GPU tests in CI.

A bug was introduced in huggingface#2797 where state.SCB was accessed while
dequantizing 4bit bnb weights even though state is None. This would
occur, for instance, when using DoRA, which needs to dequantize the
weight. The attribute access is now restricted to 8bit bnb weights.
@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.

@BenjaminBossan
BenjaminBossan merged commit 41091ec into huggingface:main Oct 27, 2025
34 of 37 checks passed
@BenjaminBossan
BenjaminBossan deleted the fix-dequantizing-4bit-bnb branch October 27, 2025 15:51
iambogeumkim pushed a commit to iambogeumkim/peft that referenced this pull request Jun 21, 2026
Fixes some failing GPU tests in CI.

A bug was introduced in huggingface#2797 where state.SCB was accessed while
dequantizing 4bit bnb weights even though state is None. This would
occur, for instance, when using DoRA, which needs to dequantize the
weight. The attribute access is now restricted to 8bit bnb weights.
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