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

[Hacktoberfest] DeepFloyd IF docstrings #5302

Closed
stevhliu opened this issue Oct 5, 2023 · 1 comment · Fixed by #5304
Closed

[Hacktoberfest] DeepFloyd IF docstrings #5302

stevhliu opened this issue Oct 5, 2023 · 1 comment · Fixed by #5304

Comments

@stevhliu
Copy link
Member

stevhliu commented Oct 5, 2023

This feature request is included in this year's Diffusers Hacktoberfest 🎃!

Hacktoberfest's purpose is to give upcoming open-source developers the opportunity to learn about open-source and programming by fixing simple issues (like this one).

How to solve the issue:

  • Every issue has a description further below explaining the problem. First, you should try to understand the issue.
  • Second, follow the contribution guide to learn how to contribute to Diffusers.
  • Claim this issue for you by opening a PR that links to this issue and writing, "I am solving this issue in ".

Issue

For each of the DeepFloyd IF pipelines:

  1. The encode_prompt parameters are all mixed up and need to be reformatted
  2. The parameter values in __call__ are out of sync (for example, num_inference_steps should be 100 but the docstring says it is 50). Please also check if other values are also incorrect, like the guidance_scale
@m0saan
Copy link
Contributor

m0saan commented Oct 5, 2023

@stevhliu I would like to work on this please!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants