Skip to content

FIX: Adjust transformers version check for bloom#1992

Merged
BenjaminBossan merged 2 commits intohuggingface:mainfrom
BenjaminBossan:fix-transformers-version-check-for-bloom
Aug 6, 2024
Merged

FIX: Adjust transformers version check for bloom#1992
BenjaminBossan merged 2 commits intohuggingface:mainfrom
BenjaminBossan:fix-transformers-version-check-for-bloom

Conversation

@BenjaminBossan
Copy link
Copy Markdown
Member

See #1969

The fix to the bloom architecture was not actually released in transformers 4.43.3, which makes the version check invalid. Instead, now checking an attribute on the BloomPreTrainedModel.

See huggingface#1969

The fix to the bloom architecture was not actually released in
transformers 4.43.3, which makes the version check invalid. Instead, now
checking an attribute on the BloomPreTrainedModel.
> is a more general check for all models, so might be more preferred for
tests
@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.

Copy link
Copy Markdown
Contributor

@gante gante left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@BenjaminBossan BenjaminBossan merged commit 4611034 into huggingface:main Aug 6, 2024
@BenjaminBossan BenjaminBossan deleted the fix-transformers-version-check-for-bloom branch August 6, 2024 11:40
Guy-Bilitski pushed a commit to Guy-Bilitski/peft that referenced this pull request May 13, 2025
The fix to the bloom architecture was not actually released in
transformers 4.43.3, which makes the version check invalid. Instead, now
checking an attribute on the BloomPreTrainedModel.
cyyever pushed a commit to cyyever/peft that referenced this pull request Sep 4, 2025
* adds cli and import utils

* updates SFT script

* adds liger model to trainer

* adds liger nightly dep

* precommit

* fix import

* Update trl/commands/cli_utils.py

* Fix quality

* moved use_liger arg to sft config

* remove arg

* remove use liger from sft trainer

---------

Co-authored-by: lewtun <lewis.c.tunstall@gmail.com>
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