search

Sygil-Dev / stable-diffusion

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

The colab launches the webui from the wrong folder #108

Closed pinilpypinilpy closed 2 years ago

pinilpypinilpy commented 2 years ago

In the colab, the final launch command is: !python /content/stable-diffusion/scripts/webui.py \ --ckpt '{models_path}/sd-v1-4.ckpt' \ --outdir '{output_path}' \ --share {vars}

Which throws an error because webui.py tries to find optimizedSD/v1-inference.yaml in the /content directory.

An easy solution is: %cd /content/stable-diffusion !python scripts/webui.py \ --ckpt '{models_path}/sd-v1-4.ckpt' \ --outdir '{output_path}' \ --share {vars}

hlky commented 2 years ago

@altryne

altryne commented 2 years ago

Yeah I saw this, it's what I get by... linking directly to a repo instead of pining a version! @hlky are we going to do released at some point ? 😂

@pinilpypinilpy this was fixed in https://github.com/altryne/sd-webui-colab/commit/d0ac994d065722c8a07742865bc0b3f0cfb4d89f If you don't mind testing again? 🙏 Thank you for the issue

altryne commented 2 years ago

@hlky this can be closed

pinilpypinilpy commented 2 years ago

Looks good, thanks!

As an aside, would it be possible to have the notebook download the model for people with slow upload speeds?

I ended up using this: https://gist.github.com/davo/4b374f41be657f59bfb60d1ee61d2616 to save myself a lot of time

alexserpico commented 2 years ago

form me it's working only changing as suggested by @pinilpypinilpy

pinilpypinilpy commented 2 years ago

form me it's working only changing as suggested by @pinilpypinilpy

Yeah I just tested it, the issue seems to be that the python command isn't on a new line, so it conflicts with cd.

altryne commented 2 years ago

ugh! that's what I get by doing this in github 😮‍💨