-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Update accelerate dependency version #892
Conversation
Thanks for finding this issue and proposing a solution. I checked and it seems that In the future, we might want to just copy these small functions into PEFT to avoid having to restrict the version of packages for just a few lines of code. |
I was on |
Yes, it was added in 0.21 AFAICT. |
Should we set the required version > 0.21 ? Or do you have a different solution? |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
@rohithkrn Would you be so kind to update the version? |
@BenjaminBossan updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
Update accelerate version to fix -