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

[Community] Support StableDiffusionCanvasPipeline #3590

Merged
merged 12 commits into from
Jun 7, 2023
Merged

[Community] Support StableDiffusionCanvasPipeline #3590

merged 12 commits into from
Jun 7, 2023

Conversation

kadirnar
Copy link
Contributor

@kadirnar kadirnar commented May 28, 2023

I will update the readme after this is merged.

#3586

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@kadirnar
Copy link
Contributor Author

kadirnar commented May 30, 2023

@patrickvonplaten , @sayakpaul

How do I import this Text2ImageRegion class? Should I add it to the pipeline_utils python file?

@sayakpaul
Copy link
Member

@patrickvonplaten , @sayakpaul

How do I import this Text2ImageRegion class? Should I add it to the pipeline_utils python file?

Sounds like a good idea!

@patrickvonplaten
Copy link
Contributor

Hey @kadirnar,

Could you also add a section to the README here: https://github.com/huggingface/diffusers/tree/main/examples/community#community-examples

@patrickvonplaten
Copy link
Contributor

Works for me!

@patrickvonplaten patrickvonplaten merged commit cd61869 into huggingface:main Jun 7, 2023
@kadirnar kadirnar deleted the add_canvas_pipeline branch June 7, 2023 16:43
@kadirnar
Copy link
Contributor Author

kadirnar commented Jun 7, 2023

@patrickvonplaten ,

Thank you, but the readme.md file should be updated. Isn't this code wrong?

from diffusers.pipelines.pipeline_utils import Image2ImageRegion, Text2ImageRegion, preprocess_image

@patrickvonplaten
Copy link
Contributor

Wanna open a quick new PR ? :-)

@kadirnar
Copy link
Contributor Author

kadirnar commented Jun 7, 2023

Wanna open a quick new PR ? :-)

Yes, I want to do it, but I don't know how to import it. Because it's not in the src/diffusers folder.

diffusers.examples.community.mixture_canvas import Image2ImageRegion, Text2ImageRegion, preprocess_image

I want to do this. But this isn't true. That's why I added it to pipeline_utils.py. How should I do?

@patrickvonplaten
Copy link
Contributor

Ah yeah, I'd add all these "Region" classes directly in the same file so that you don't have to import it

AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* added StableDiffusionCanvasPipeline pipeline

* Added utils codes to pipe_utils file.

* make style

* delete mixture.py and Text2ImageRegion class

* make style

* Added the codes to the readme.md file.

* Moved functions from pipeline_utils to mix_canvas
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.

4 participants