-
Notifications
You must be signed in to change notification settings - Fork 316
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
Model config tests #627
base: dev
Are you sure you want to change the base?
Model config tests #627
Conversation
The failure is due to these tests now triggering http requests on gated models. GitHub does allow for two buckets of variables, and we could put the hf token into the variables section to make this work, but that would involve also allowing access to secrets, which is not possible. It seems like a lot of people are requesting this sort of functionality though, so maybe a solution will come up later https://github.com/orgs/community/discussions/44322. The other solution would be to manually drop on the config for these models, which is very feasible. |
Description
Added a test to verify that all model configs currently listed in loading_from_pretrained are not None.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
Screenshots
Please attach before and after screenshots of the change if applicable.
Checklist: