Sygil-Dev / sygil-webui

Stable Diffusion web UI
GNU Affero General Public License v3.0
7.85k stars 889 forks source link

Adding back deleted docker-compose.yml #1825

Closed DmitrySychev closed 7 months ago

DmitrySychev commented 7 months ago

Description

Please include: I'm not sure why this docker-compose.yml was deleted in the first place. Nearly half of the instructions in the readme for this repo rely on this file being there.

Essentially, it's just another way to start up the container using docker compose . This makes the container much more usable for power users, as they'd be able to configure volumes much easier. Also, instead of having to remember the whole complicated docker container run --rm -d -p 8501:8501 -e STREAMLIT_SERVER_HEADLESS=true -e "WEBUI_SCRIPT=webui_streamlit.py" -e "VALIDATE_MODELS=false" -v "${PWD}/outputs:/sd/outputs" --gpus all tukirito/sygil-webui:latest command, it's as simple as running docker compose start or up, since all of the configs are in the compose file, you wouldn't need anything else. The compose file has been tested and runs without any issues. The readme instructions can stay mostly as is.

Additionally, as mentioned before, the readme instructions rely heavily on docker compose. If the repo creators wish to not include docker compose, then it should all be omitted from the readme, as it makes this extremely confusing for any user.

Closes: # (issue)

Checklist:

yiqiangjun commented 7 months ago

这是来自QQ邮箱的假期自动回复邮件。   您好,我最近正在休假中,无法亲自回复您的邮件。我将在假期结束后,尽快给您回复。

sweep-ai[bot] commented 7 months ago

Apply Sweep Rules to your PR?

ZeroCool940711 commented 7 months ago

Hey, thanks for the PR, a lot of people have been asking for this but honestly I'm not good with docker so I was not able to fix it myself, really appreciate your PR. I think we should first merge it to the dev branch and then to main so all the changes in the dev branch are on main. I will be changing the bran on this PR to the dev branch and then merging it and also merging dev to main afterward.