-
Notifications
You must be signed in to change notification settings - Fork 736
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
Add model file name #594
Add model file name #594
Conversation
Thanks for the PR! Just to keep the diff small, can you remove any whitespace formatting applied by your editor? There definitely are valid changes, but it will make the diff large for the PR. 🤗 |
@xenova Absolutely. Done. Thanks for the suggestion. The PR now has two lines changed. Much cleaner! The PR is also merged with 2.16.0 (released a few hours ago), and is ready to go :) |
The PR is merged with 2.16.1 and is ready to go :) |
@xenova Those changes are already in v3. This PR should be merge into main, or close until v3 arrives |
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!
nits applied to keep future diffs small
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
I'll make the necessary changes in v3 myself. For now, this is okay (and will work for all single-file models, like encoder-only and decoder-only models). |
No description provided.