-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Update requirements.txt #3040
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 requirements.txt #3040
Conversation
|
Thanks for your contribution! |
requirements.txt
Outdated
| tokenizers<=0.20.3; python_version<="3.8" | ||
| tokenizers>=0.21,<0.22; python_version>"3.8" | ||
| tokenizers; python_version<="3.8" | ||
| tokenizers>=0.21; python_version>"3.8" |
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.
transformers本身依赖tokenizers,可以不写这个依赖
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.
跟随transformers对tokenizers的版本限制就行
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3040 +/- ##
==========================================
Coverage ? 34.08%
==========================================
Files ? 428
Lines ? 69526
Branches ? 0
==========================================
Hits ? 23697
Misses ? 45829
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
lugimzzz
left a comment
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
解除multiprocess,numpy,tokenizers依赖的版本上限