-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Standardize model card for textual inversion sdxl #6963
Standardize model card for textual inversion sdxl #6963
Conversation
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. |
This should be good to merge now @sayakpaul |
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.
Should be good to merge after the styling problems have been fixed. I have left a minor suggestion as well.
Ok, I cleaned that all up! I appreciate all of your input on this @sayakpaul, I mistakenly ran isort against the file and it shuffled around the formatting of all the imports. I should've just ran |
The code quality is still not resolved. I am afraid if that is not resolved, we cannot merge the PR. You need to run
|
…://github.com/Stepheni12/diffusers into standardize-model-card-textual-inversion-sdxl
Hmm that's odd, I copied over the imports from the current version of the file on the main branch so not sure why it was failing the tests if that is the code already on main. Either way, I ran |
Nice code quality tests are passing now. Once the CI run is complete, will merge :) |
Thanks for your contributions! |
What does this PR do?
In reference to: #6891
Standardizes textual inversion sdxl model card
Notebook link here
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
@sayakpaul