search

Sygil-Dev / stable-diffusion

GNU Affero General Public License v3.0
1.72k stars 148 forks source link

Dockerfile Missing pynvml #126

Closed bluedevil678 closed 2 years ago

bluedevil678 commented 2 years ago

I had to include this in the latest docker file:

RUN pip install pynvml

This was required to allow for launch (30/08/22) - Works fine one added.

hlky commented 2 years ago

Will leave this for the Docker maintainer, please include a proper title in your issues.

yourjelly commented 2 years ago

It should be covered by the environment.yaml

https://github.com/hlky/stable-diffusion/blob/d4fc920b1e67abdcdb2d9ca95ce2890e849752d3/environment.yaml#L31

Does it really need to be run in the Dockerfile?

bluedevil678 commented 2 years ago

Indeed it could be covered there, just provide one possible option ;)

On Tue, Aug 30, 2022, 17:38 Jon Xuereb @.***> wrote:

It should be covered by the environment.yaml

https://github.com/hlky/stable-diffusion/blob/d4fc920b1e67abdcdb2d9ca95ce2890e849752d3/environment.yaml#L31

Does it really need to be run in the Dockerfile?

— Reply to this email directly, view it on GitHub https://github.com/hlky/stable-diffusion/issues/126#issuecomment-1231907261, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU4BOYQQ4J6TX7O2HE2NUC3V3Y2H7ANCNFSM6AAAAAAQACYWUE . You are receiving this because you authored the thread.Message ID: @.***>

oc013 commented 2 years ago

It already updates on its own, just restart the container...

$ docker compose up
[+] Running 1/0
 ⠿ Container sd  Created                                                                                                                                             0.0s
Attaching to sd
sd  | Updating conda env: ldm ...