Skip to content

Conversation

@linlifan
Copy link
Contributor

@linlifan linlifan commented Nov 2, 2023

…formance

What does this PR do?

fix an issue that when enable IPEX (especial ipex 2.0) host memory occupied to 80GB for 896x896 . after this fix, it will reduce to 20GB.

Fixes # (issue)

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@linlifan
Copy link
Contributor Author

linlifan commented Nov 2, 2023

@patrickvonplaten and @sayakpaul could you please help to review? thank you!

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Nov 2, 2023

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

@yiyixuxu
Copy link
Collaborator

yiyixuxu commented Nov 3, 2023

@linlifan
can you run make style so the test pass?

@linlifan
Copy link
Contributor Author

@linlifan can you run make style so the test pass?

@yiyixuxu thank you ! and addressed.

@patrickvonplaten patrickvonplaten merged commit 4abbbff into huggingface:main Nov 20, 2023
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
huggingface#5625)

* fix an issue that ipex occupy too much memory, it will not impact performance

* make style

---------

Co-authored-by: root <[email protected]>
Co-authored-by: Meng Guoqing <[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.

5 participants