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

Fixed filepaths for model files in configs/prompts/animation.yaml #10

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

Conversation

MaximilianKr
Copy link

Problem:
The animation.yaml did not have the correct paths to the model files, hence one encountered an error message about an incorrect repo_id.

Solution:
Prepend "magicanimate/" to the paths to reflect the correct folder structure.

@MaximilianKr MaximilianKr changed the title Fixed filepaths for model files in configs/promtps/animation.yaml Fixed filepaths for model files in configs/prompts/animation.yaml Dec 4, 2023
@MaximilianKr MaximilianKr mentioned this pull request Dec 4, 2023
@xiohulp
Copy link

xiohulp commented Dec 22, 2023

followed the instruction as you told, but still get error messages as:
Traceback (most recent call last):
File "C:\Users\xiohu.conda\envs\python310\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\xiohu.conda\envs\python310\lib\runpy.py", line 86, in _run_code
exec(code, run_globals)
File "D:\magicanimate\demo\gradio_animate.py", line 19, in
animator = MagicAnimate()
File "D:\magicanimate\demo\animate.py", line 58, in init
tokenizer = CLIPTokenizer.from_pretrained(config.pretrained_model_path, subfolder="tokenizer")
File "C:\Users\xiohu.conda\envs\python310\lib\site-packages\transformers\tokenization_utils_base.py", line 1813, in from_pretrained
resolved_vocab_files[file_id] = cached_file(
File "C:\Users\xiohu.conda\envs\python310\lib\site-packages\transformers\utils\hub.py", line 428, in cached_file
resolved_file = hf_hub_download(
File "C:\Users\xiohu.conda\envs\python310\lib\site-packages\huggingface_hub\utils_validators.py", line 110, in _inner_fn
validate_repo_id(arg_value)
File "C:\Users\xiohu.conda\envs\python310\lib\site-packages\huggingface_hub\utils_validators.py", line 158, in validate_repo_id
raise HFValidationError(
huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': 'magicanimate/pretrained_models/stable-diffusion-v1-5'. Use repo_type argument if needed.

@MaximilianKr
Copy link
Author

I'm not sure the fix was necessary for everyone as the pr never got merged and they still have the original version in their repo.
It was necessary for me on W11/WSL.
Did you encounter the same error before changing the filepaths? What system?

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