-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Add support for CodeLlama #854
Conversation
Signed-off-by: Antoni Baum <[email protected]>
@WoosukKwon @zhuohan123 I think this can be merged now! |
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! Thanks for the quick PR and left a small comment. BTW does this changes our requirement for the version of the transformers
library?
Co-authored-by: Zhuohan Li <[email protected]>
@zhuohan123 The code is backwards compatible, but in order for someone to use CodeLlama with longer context, they will need to install (as of yet unreleased) transformers==4.33.0 |
Sounds good! Let me merge this PR first and we can bump up the requirements once |
Needs huggingface/transformers#25740 to land first