-
Notifications
You must be signed in to change notification settings - Fork 484
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
Unable to build diffusion
containers after recent updates to opencv
#489
Comments
Hi, I have the same issue. @ms1design where you able to solve it or work around it? |
Unfortunately, no, but I haven't tried yet tbh. I have this on my to-do list for when I have more time available. |
I think I fixed it in #541 |
Thanks @ai-and-i , took a deeper look and fixed it another way in e0368bf It was related to the CUDA-enabled opencv wheels being called |
Perfect, thanks for the quick fix! |
Fixed in e0368bf, closing. |
Hi @dusty-nv,
Just taking a note that we had a breaking change in
opencv
that negatively impacted thediffusion
containers breaking the builds:In
opencv
we have new way of install/building:jetson-containers/packages/opencv/Dockerfile
Lines 25 to 29 in 487dc4f
But this fails hard on
stable-diffusion
container whereinvisible-watermark
dependency requiresopencv-python
and cannot find proper distribution.Same with
stable-diffusion-webui
:jetson-containers/packages/diffusion/stable-diffusion-webui/Dockerfile
Lines 35 to 36 in 487dc4f
Started to fix that in hope it's a quick one – running the
opencv
install/build sh scripts, but I gave up when realised that it starts thebuild.sh
fromopencv
.build.sh
requires the unavailable environment variables (accessible only when buildingopencv
cotnainer):jetson-containers/packages/opencv/config.py
Lines 9 to 12 in 487dc4f
The text was updated successfully, but these errors were encountered: