Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI Update AutoAWQ version to fix CI #2222

Conversation

BenjaminBossan
Copy link
Member

Currently, the CI fails because awq tries to import a non-existing function from transformers (presumably it was there at one point but no longer is):

from transformers.modeling_utils import shard_checkpoint

E ImportError: cannot import name 'shard_checkpoint' from 'transformers.modeling_utils' (/opt/conda/envs/peft/lib/python3.11/site-packages/transformers/modeling_utils.py)

This has been fixed in awq v0.2.7. Therefore, this newer version is now used in CI.

Currently, the CI fails because awq tries to import a non-existing
function from transformers (presumably it was there at one point but no
longer is):

>     from transformers.modeling_utils import shard_checkpoint
> E   ImportError: cannot import name 'shard_checkpoint' from 'transformers.modeling_utils' (/opt/conda/envs/peft/lib/python3.11/site-packages/transformers/modeling_utils.py)

This has been fixed in awq v0.2.7. Therefore, this newer version is now
used in CI.
@HuggingFaceDocBuilderDev

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
Member

@SunMarc SunMarc left a comment

Choose a reason for hiding this comment

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

Thanks for the update @BenjaminBossan !

@SunMarc SunMarc requested a review from LysandreJik November 19, 2024 15:36
@BenjaminBossan BenjaminBossan merged commit 8874ab5 into huggingface:main Nov 19, 2024
16 checks passed
@BenjaminBossan
Copy link
Member Author

Oh, I merged before noticing you requested a review from Lysandre. Not sure if that's necessary for such a small change.

@BenjaminBossan BenjaminBossan deleted the fix-docker-update-auto-awq-version branch November 19, 2024 15:54
@LysandreJik
Copy link
Member

no problem for me!

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