Skip to content

[docs] More API fixes #3640

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

Merged
merged 5 commits into from
Jun 5, 2023
Merged

Conversation

stevhliu
Copy link
Member

@stevhliu stevhliu commented Jun 1, 2023

Adds the following API fixes:

  • Return statements to the download, from_config, and load_config functions
  • Move ImagePipelineOutput, AudioPipelineOutput, and ImageTextPipelineOutput to the Outputs page
  • Moves non "Stable Diffusion" pipelines out of the section (included SafeStableDiffusion here)
  • Create new Utilities API page for randn_tensor
  • Clarify parameters in docstring

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Jun 1, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Magician there 🪄

Copy link
Contributor

@patrickvonplaten patrickvonplaten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool!

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing work, thanks a lot!

Comment on lines +9 to +15
## numpy_to_pil

[[autodoc]] utils.pil_utils.numpy_to_pil

## pt_to_pil

[[autodoc]] utils.pil_utils.pt_to_pil
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these are usually accessed as static functions of the pipelines, which is usually convenient. Not sure if we should cross-reference them somehow (not necessary for this PR, just to keep in mind).

@stevhliu stevhliu merged commit 1a6a647 into huggingface:main Jun 5, 2023
@stevhliu stevhliu deleted the stable-diffusion-api branch June 5, 2023 16:47
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
* part 2 of api fixes

* move randn_tensor

* add to toctree

* apply feedback

* more feedback
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* part 2 of api fixes

* move randn_tensor

* add to toctree

* apply feedback

* more feedback
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.

5 participants