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

Force huggingface_hub version to prevent breaking change #96

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gabfl
Copy link

@gabfl gabfl commented Oct 19, 2024

On 10/17/24, hugging face released version v0.26.0 of huggingface_hub which introduced the following breaking changes:

Screenshot 2024-10-18 at 9 47 36 PM

This creates a stacktrace because diffusers pulls the latest version of huggingface_hub as seen here.

Full stack trace:

$ python inference.py --inference_config configs/test.yaml
Traceback (most recent call last):
  File "MimicMotion/inference.py", line 16, in <module>
    from mimicmotion.utils.geglu_patch import patch_geglu_inplace
  File "MimicMotion/mimicmotion/utils/geglu_patch.py", line 1, in <module>
    import diffusers.models.activations
  File "MimicMotion/venv/lib/python3.11/site-packages/diffusers/__init__.py", line 5, in <module>
    from .utils import (
  File "MimicMotion/venv/lib/python3.11/site-packages/diffusers/utils/__init__.py", line 38, in <module>
    from .dynamic_modules_utils import get_class_from_dynamic_module
  File "MimicMotion/venv/lib/python3.11/site-packages/diffusers/utils/dynamic_modules_utils.py", line 28, in <module>
    from huggingface_hub import cached_download, hf_hub_download, model_info
ImportError: cannot import name 'cached_download' from 'huggingface_hub' (MimicMotion/venv/lib/python3.11/site-packages/huggingface_hub/__init__.py)

This pull requests forces the version 0.25.2 of huggingface_hub which resolves the stacktrace.

@QQbbla
Copy link

QQbbla commented Nov 13, 2024

pip install huggingface_hub==0.24.7

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