a few fix for kandinsky combined pipeline by yiyixuxu · Pull Request #4352 · huggingface/diffusers · GitHub
Skip to content

a few fix for kandinsky combined pipeline - #4352

Merged
yiyixuxu merged 4 commits into
mainfrom
combined-pipeline-xformer
Aug 4, 2023
Merged

yiyixuxu merged 4 commits into
mainfrom
combined-pipeline-xformer

Conversation

@yiyixuxu

Copy link
Copy Markdown
Collaborator

this PR:

  • add enable_sequential_cpu_offload and enable_xformers_memory_efficient_attention to combined pipelines
  • test xformer for combined pipelines

should fix
#4332

@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Jul 29, 2023 •

Copy link
Copy Markdown

Comment thread src/diffusers/pipelines/kandinsky/pipeline_kandinsky_combined.py Outdated
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

@patrickvonplaten patrickvonplaten left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great!

@yiyixuxu
yiyixuxu merged commit 29ece0d into main Aug 4, 2023
@yiyixuxu
yiyixuxu deleted the combined-pipeline-xformer branch August 4, 2023 01:10
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
* add xformer

* enable_sequential_cpu_offload

* style

* Update src/diffusers/pipelines/kandinsky/pipeline_kandinsky_combined.py

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

---------

Co-authored-by: yiyixuxu <yixu310@gmail,com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* add xformer

* enable_sequential_cpu_offload

* style

* Update src/diffusers/pipelines/kandinsky/pipeline_kandinsky_combined.py

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

---------

Co-authored-by: yiyixuxu <yixu310@gmail,com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
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