Dockerfile Missing pynvml

[bluedevil678]

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]

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

[yourjelly]

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]

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 <#126 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AU4BOYQQ4J6TX7O2HE2NUC3V3Y2H7ANCNFSM6AAAAAAQACYWUE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[oc013]

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