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

Prompt2Prompt: Inherit from DiffusionPipeline #7211

Conversation

nxbringr
Copy link
Contributor

@nxbringr nxbringr commented Mar 4, 2024

What does this PR do?

  • Refactors Prompt2PromptPipeline class to inherit from DiffusionPipeline instead of StableDiffusionPipeline class.
  • Adds (copies) dependencies from pipeline_stable_diffusion
  • Tested it against a few examples mentioned in community README here.

Fixes #6984

Before submitting

Who can review?

Apologies for the delay. Looking forward for your review :)
@yiyixuxu

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.

Very nice changes. Thank you!

@yiyixuxu good example where we should check if the "# Copied from ... statements" are taking effect.

@sayakpaul sayakpaul requested a review from yiyixuxu March 5, 2024 11:12
@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.

Copy link
Collaborator

@yiyixuxu yiyixuxu left a comment

Choose a reason for hiding this comment

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

thank you:)

@yiyixuxu yiyixuxu merged commit 9940573 into huggingface:main Mar 7, 2024
8 checks passed
@nxbringr nxbringr changed the title Refactor Prompt2Prompt: Inherit from DiffusionPipeline Prompt2Prompt: Inherit from DiffusionPipeline Mar 19, 2024
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.

refactors pipelines inheriting from StableDiffusionPipeline
4 participants