Skip to content
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

Merged
merged 19 commits into from
Oct 21, 2024

Conversation

jeongiin
Copy link
Contributor

@jeongiin jeongiin commented Oct 5, 2024

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

Who can review?

@a-r-r-o-w @stevhliu

Copy link
Member

@stevhliu stevhliu left a 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 :)

@HuggingFaceDocBuilderDev

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.

@a-r-r-o-w
Copy link
Member

Hi @jeongiin, just need to run make style and push changes here to be ready for merge I think

@jeongiin
Copy link
Contributor Author

jeongiin commented Oct 9, 2024

Hello @a-r-r-o-w and @stevhliu I fixed style!

@stevhliu
Copy link
Member

stevhliu commented Oct 9, 2024

Thanks! Can you also run make fix-copies to align all the # Copied from statements?

@jeongiin
Copy link
Contributor Author

Thanks for catching that!! @stevhliu

@stevhliu
Copy link
Member

Hmm, the copy inconsistencies are still there for some reason. Can you try running python utils/check_copies.py --fix_and_overwrite instead?

@jeongiin
Copy link
Contributor Author

I've tried running python utils/check_copies.py --fix_and_overwrite, but nothing happens to the problematic file. Could you suggest another solution? :(

@stevhliu
Copy link
Member

Hmm, I'm not entirely sure then! Running those commands and pushing their changes should fix the tests. @sayakpaul do you have another suggestion?

@sayakpaul
Copy link
Member

We need to apply make fix-copies to fix the errors.

@a-r-r-o-w
Copy link
Member

@stevhliu @sayakpaul good to merge?

@sayakpaul
Copy link
Member

Okay for me!

@stevhliu stevhliu merged commit bcd61fd into huggingface:main Oct 21, 2024
15 checks passed
@HanLiii
Copy link

HanLiii commented Oct 25, 2024

Will pipeline_animatediff_sdxl_video2video.py be released in the future?

@a-r-r-o-w
Copy link
Member

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 :)

sayakpaul added a commit that referenced this pull request Dec 23, 2024
* 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[community] Improving docstrings and type hints
6 participants