Skip to content
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

Installation Issues #41

Closed
bitshifter52 opened this issue Mar 9, 2023 · 3 comments · Fixed by #42
Closed

Installation Issues #41

bitshifter52 opened this issue Mar 9, 2023 · 3 comments · Fixed by #42

Comments

@bitshifter52
Copy link

I followed the readme instructions and things are not working.

I used "https://github.com/yownas/seed_travel.git" in the extensions tab and it shows up in the list of installed extensions. I also added the line "moviepy==1.0.3" to "requirements_versions.txt". I tried the install: !pip install moviepy==1.0.3 and it doesn't work, stable-diffusion-webui does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

When I search for pyproject.toml in the stable-diffusion-webui folder it appears in several places:
/stable-diffusion-webui/repositories/k-diffusion/pyproject.toml
/stable-diffusion-webui/extensions/sd-dynamic-prompts/pyproject.toml
/stable-diffusion-webui/venv/lib/python3.9/site-packages/spacy/tests/package/pyproject.toml
/stable-diffusion-webui/repositories/xformers/third_party/cutlass/tools/library/scripts/pycutlass/pyproject.toml

A search for setup.py in the stable-diffusion-webui folder returns too many instances to list here.

Now when I run ./webui.sh I receive the following messages:
Launching Web UI with arguments: --opt-split-attention --xformers
Error loading script: seed_travel.py
Traceback (most recent call last):
File "/home/{whoami}/stable-diffusion-webui/modules/scripts.py", line 229, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "/home/{whoami}/stable-diffusion-webui/modules/script_loading.py", line 11, in load_module
module_spec.loader.exec_module(module)
File "", line 846, in exec_module
File "", line 983, in get_code
File "", line 913, in source_to_code
File "", line 228, in _call_with_frames_removed
File "/home/robert/stable-diffusion-webui/scripts/seed_travel.py", line 8

^
SyntaxError: invalid syntax

At this point I don't know how to proceed - so any suggestions will be appreciated.

@yownas
Copy link
Owner

yownas commented Mar 9, 2023

If you installed ir through the extensions tab there is no need to add anything to requirements_versions.txt or install by hand. setup.py is used by the webui to install things, nothing you should need to run.

I'm a little but confused, it looks like you have it installed in the scripts folder, but it should have ended up under extensions.

I think the best thing would be to remove seed_travel.py from the scripts folder. And then install it from the extensions-tab; click Available, then Load from, search for "seed_travel" and click Install.

@bitshifter52
Copy link
Author

Thanks for the response, I'll try your suggestion and see what happens.

neggles added a commit to neggles/seed_travel that referenced this issue Mar 15, 2023
Fixes yownas#41 by adding imageio-ffmpeg to extension requirements/dependencies.

This is required due to some changes with moviepy's underlying imageio
library, which has deprecated/removed the old autoinstallation method
around 2019-ish, and as such moviepy is no longer capable of
autoinstalling ffmpeg.

See Zulko/moviepy#906 and Zulko/moviepy#908
for further details on the moviepy side.

This does result in downloading ffmpeg on systems where it may already
be installed, but it's a reasonably small download and on some systems
(eg Ubuntu 22.04) moviepy 1.0.3 fails to pick up system ffmpeg anyway.
@neggles
Copy link
Contributor

neggles commented Mar 15, 2023

Seems imageio-ffmpeg is required for moviepy to work properly; even after installing ffmpeg on an Ubuntu 22.04 image (docker container) the import moviepy.video.io.ImageSequenceClip as ImageSequenceClip line fails

Looking again this may not actually be directly related to bitshifter52's issue 😅 but it has been causing me trouble...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants