-
Notifications
You must be signed in to change notification settings - Fork 769
Task: transformers 5 support #1582
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
base: main
Are you sure you want to change the base?
Task: transformers 5 support #1582
Conversation
Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>
Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>
Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>
de2ea96 to
7cc3c4e
Compare
huggingface_hub version 1.0 changes the entry point from `huggingface_cli` to `hf` Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>
erickgalinkin
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.
Looks good to me -- support merging once all tests pass.
leondz
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.
Thanks. Good luck w/ win.
cafaddf to
6b96d0a
Compare
Currently the custom generator used fails to load in windows, allow exception to be raised Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>
6b96d0a to
44e9a86
Compare
| if sys.platform == "win32" and klassname == "buffs.paraphrase.Fast": | ||
| # special case buff not currently supported on Windows | ||
| with pytest.raises(GarakException) as exc_info: | ||
| list(b.transform(a)) # process yield to see raise | ||
| assert "failed" in str(exc_info.value) |
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.
Note this test should start to fail if/when Windows reaches a working state for this plugin.
erickgalinkin
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
Update for transformers major version release.
Verification
List the steps needed to make sure this thing works