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

Fix installation by pinning diffusers version to 0.9.0 #163

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

veeral-patel
Copy link
Contributor

@veeral-patel veeral-patel commented Nov 30, 2023

Thanks for this great project!

TLDR: Pin diffusers to 0.9.0 to fix install errors for new devs.
This should solve issues #161 #162 #160 #152
Similar to PRP #158, but that PR contains some auxiliary changes making CI fail

Problem

I reproduced not able to get my server running locally, with a fresh Riffusion download. I followed these steps:

virtualenv venv && source venv/bin/activate
pip install -r requirements.txt

And got this error:

  File "/Users/veeralpatel/dev/riffusion2/riffusion/riffusion_pipeline.py", line 14, in <module>
    from diffusers.pipeline_utils import DiffusionPipeline
ModuleNotFoundError: No module named 'diffusers.pipeline_utils'

Solution

I noticed pip is installing diffusers 0.24.0. I browsed the diffusers code. Looks like they refactored. We need to import DiffusionPipeline like this now:

from diffusers import DiffusionPipeline
# or
from diffusers.pipelines import DiffusionPipeline

I've just pinned diffusers to 0.9.0 which gets my server running fine.

(venv) riffusion git:main ❯ python -m riffusion.server --host 127.0.0.1 --port 3013                       ✹
/Users/veeralpatel/dev/riffusion3/riffusion/server.py:55: UserWarning: WARNING: cuda is not available, using cpu instead.
  PIPELINE = RiffusionPipeline.load_checkpoint(
WARNING: Falling back to float32 on cpu, float16 is unsupported
Fetching 15 files:   0%|                                                             | 0/15 [00:00<?, ?it/s^Fetching 15 files:  20%|██████████▌                                          | 3/15 [00:01<00:07,  1.57it/s]

@veeral-patel veeral-patel changed the title Pin diffusers version to 0.9.0 Fix installation by pinning diffusers version to 0.9.0 Nov 30, 2023
@hmartiro hmartiro merged commit 70fb80f into riffusion:main Nov 30, 2023
5 of 7 checks passed
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 this pull request may close these issues.

2 participants