Skip to content

Update to transformers 4.28 - #10

Open
ntoxeg wants to merge 1 commit into
kuleshov:mainfrom
ntoxeg:main
Open

ntoxeg wants to merge 1 commit into
kuleshov:mainfrom
ntoxeg:main

Conversation

@ntoxeg

@ntoxeg ntoxeg commented Apr 21, 2023

Copy link
Copy Markdown

transformers have LLaMA officially implemented since version 4.28.
This will raise some warning about incongruent types but it should be fine for actual use.

transformers have LLaMA officially
implemented since version 4.28.
@kuleshov

Copy link
Copy Markdown
Owner

thank you! i'll test this out myself soon, but just want to check if you did

@ntoxeg

ntoxeg commented May 15, 2023

Copy link
Copy Markdown
Author

Yes, although just in case make sure to test the CLI, as I only used the programmatic interface (except for downloading).
You will get some warnings that class names don’t match (because it’s different capitalization), but otherwise it worked fine.

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.

2 participants