-
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
[docs] add docstrings in pipline_stable_diffusion.py
#9590
[docs] add docstrings in pipline_stable_diffusion.py
#9590
Conversation
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.
Nice job!
Run make style
to automatically format the code so the CI test passes :)
src/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py
Outdated
Show resolved
Hide resolved
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. |
…sion.py Co-authored-by: Steven Liu <[email protected]>
Hi @jeongiin, just need to run |
Hello @a-r-r-o-w and @stevhliu I fixed style! |
Thanks! Can you also run |
Thanks for catching that!! @stevhliu |
Hmm, the copy inconsistencies are still there for some reason. Can you try running |
I've tried running |
Hmm, I'm not entirely sure then! Running those commands and pushing their changes should fix the tests. @sayakpaul do you have another suggestion? |
We need to apply |
@stevhliu @sayakpaul good to merge? |
Okay for me! |
Will pipeline_animatediff_sdxl_video2video.py be released in the future? |
Hi @HanLiii, we plan to make several improvements with #9672, which should allow creating arbitrary pipelines easily. We're ideating about the best way to move forward with the design that would allow this, so if you have any suggestions regarding that, it'd be awesome. For now, I think everyone on the team is occupied so we will not be able to commit to creating such a pipeline. If you would be interested in contributing a pipeline for it, we'd be happy to review and merge :) |
* fix the issue on flux dreambooth lora training * update : origin main code * docs: update pipeline_stable_diffusion docstring * docs: update pipeline_stable_diffusion docstring * Update src/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py Co-authored-by: Steven Liu <[email protected]> * fix: style * fix: style * fix: copies * make fix-copies * remove extra newline --------- Co-authored-by: Steven Liu <[email protected]> Co-authored-by: Aryan <[email protected]> Co-authored-by: Sayak Paul <[email protected]>
What does this PR do?
Fixes : #9567
I have added a docstring to the
rescale_noise_cfg
function. I plan to continue addressing this issue, and I would greatly appreciate any advice you could offer on writing effective commit messages or PRs. Thank you!ps. I've worked on other tasks as well, so there are many commit messages, but only one file was actually changed.
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
@a-r-r-o-w @stevhliu