-
Notifications
You must be signed in to change notification settings - Fork 48
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
chore: merge set of changes for v2.1.0 #376
Conversation
Signed-off-by: Anh Uong <[email protected]>
Signed-off-by: Anh Uong <[email protected]>
Signed-off-by: Harikrishnan Balagopal <[email protected]>
…er unit testing. (#348) * cleanup: Move tokenizer_data_utils to /utils to /data, change imports Signed-off-by: Will Johnson <[email protected]> * tests: Add additional tests for test_embedding_resize to check resize with special tokens, resize multiple of. fmt Signed-off-by: Will Johnson <[email protected]> * lint Signed-off-by: Will Johnson <[email protected]> * fix: more thorough testing from output of function Signed-off-by: Will Johnson <[email protected]> * test: move assertion Signed-off-by: Will Johnson <[email protected]> --------- Signed-off-by: Will Johnson <[email protected]>
Signed-off-by: Anh Uong <[email protected]>
….14 (#354) Updates the requirements on [peft](https://github.com/huggingface/peft) to permit the latest version. - [Release notes](https://github.com/huggingface/peft/releases) - [Commits](huggingface/peft@v0.8.0...v0.13.0) --- updated-dependencies: - dependency-name: peft dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Anh Uong <[email protected]>
) * deps: Upgrade accelerate to version 1.0.0 Signed-off-by: Will Johnson <[email protected]> * tests: Replace out of date variable with ,fixing unit tests Signed-off-by: Will Johnson <[email protected]> * fix: Add old parameter for backwards compatibility Signed-off-by: Will Johnson <[email protected]> --------- Signed-off-by: Will Johnson <[email protected]>
Signed-off-by: Will Johnson <[email protected]> Co-authored-by: Anh Uong <[email protected]>
Signed-off-by: Anh Uong <[email protected]>
Thanks for making a pull request! 😃 |
Thanks for making a pull request! 😃 |
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.
Looks like you're missing the most recent change from today to set upperbound of torch
deps: torch<2.5 due to FA2 error with new version
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.
LGTM
Description of the change
Related issue number
How to verify the PR
Was the PR tested