Skip to content

Conversation

@iamzoltan
Copy link
Contributor

@iamzoltan iamzoltan commented Aug 7, 2024

This is an attempt to add enable_vae_slicing and enable_vae_tiling to pipeline_flux. I have tested this locally and it seems to work. Looking to add flux to our roster of models, and this functionality would help.

@yiyixuxu and @asomoza

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.

thanks!

maybe we should have a DITMixin now? cc @sayakpaul @DN6

@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.

@sayakpaul
Copy link
Member

@yiyixuxu agreed!

@iamzoltan thanks for your contributions here. Did you notice any particular scenarios where enabling slicing and tiling have been helpful (perhaps when operating at higher resolutions on consumer GPU cards)?

@iamzoltan
Copy link
Contributor Author

iamzoltan commented Aug 19, 2024

@sayakpaul my pleasure! It was helpful on consumer GPU cards. We rely on slicing and tiling for other pipelines from diffusers

@yiyixuxu yiyixuxu merged commit 803e817 into huggingface:main Aug 19, 2024
sayakpaul added a commit that referenced this pull request Dec 23, 2024
add vae slicing and tiling to flux pipeline

Co-authored-by: Sayak Paul <[email protected]>
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