-
Notifications
You must be signed in to change notification settings - Fork 319
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
set python tests to transformers 4.35 #1551
Conversation
@homink so obviously this is the tranformers version which trigger the issue with wav2vec. |
Transformers version 4.35.2 still works fine for wav2vec2 testing. Can you guess what other package should i test further?
|
you seem to use torch 2.0.1 |
I ran python/tests/test_transformers.py but don't see any errors. I have torch 2.0.1+cu117 and CTranslate2 3.21.0. If this wav2vec2 testing really bothers, I don't mind removing it. Let me know how we could proceed.
|
you keep testing with a set up that is not the test config set up. what you need to do is to use the current set up, which is in the detailed log of the failing test. torch==2.1.0, ctranslate2==master, transformers==4.35 |
Thanks for the comment. Now I see what went wrong. Saving & loading models methods are changed and doesn't support my script. I will see what I can do and will get back. |
OK. The following change will work. Can you apply these into your PR? CTranslate2/python/tests/test_transformers.py Line 987 in c5f46a3
should be
CTranslate2/python/tests/test_transformers.py Line 993 in c5f46a3
should be
|
No description provided.