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

albumentations==1.4.3 #30

Merged
merged 2 commits into from
May 23, 2024
Merged

albumentations==1.4.3 #30

merged 2 commits into from
May 23, 2024

Conversation

w-e-w
Copy link

@w-e-w w-e-w commented Apr 18, 2024

recent Upstream package changes causes installing insightface break webui

fix solution, install albumentations==1.4.3 before insightface
this way pip won't auto install the newest version 1.4.4

note I did not specifically test this with this extension this PR is based off knowledge


also in webui 1.8

basicsr==1.4.2
realesrgan==0.3.0

was removed

@sebaxakerhtc
Copy link

@w-e-w
Should I also add this to my fork of the latest facefusion?
For now it works fine without it on latest WebUI

@w-e-w
Copy link
Author

w-e-w commented May 18, 2024

Should I also add this to my fork of the latest facefusion?
For now it works fine without it on latest WebUI

@sebaxakerhtc
yes starting albumentations==1.4.4 adds a new dedependency pydantic>=2.6.4 whitch is incompatible with the current version of webui

if you already installed other extension that already installs insightface with the fix I made albumentations==1.4.3 (like example control-net) then this issue won't trigger as it is already fixed by other extensions

but this issue will trigger if you are on a clean venv without albumentations and insightface and your extentions is the first one that tries to installs insightface withour pinning albumentations==1.4.3 it will blow up

for more details read the issue post I made

@sebaxakerhtc
Copy link

sebaxakerhtc commented May 18, 2024

use installed other extension that already installs

Yeah, as I remember I found this fix by myself and fixed it in demofusion or facefusion
Ok, thank you!

@linjiX linjiX merged commit f84fed7 into diffus-me:main May 23, 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.

None yet

3 participants