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

Dependencies Error #7

Closed
emi-dm opened this issue Jan 31, 2024 · 4 comments · Fixed by #8
Closed

Dependencies Error #7

emi-dm opened this issue Jan 31, 2024 · 4 comments · Fixed by #8

Comments

@emi-dm
Copy link

emi-dm commented Jan 31, 2024

Hi guys! Can you update the requirements file with the correct dependencies? I tried to use the model, but it didn't load the weights correctly. Can you test it?

Thanks in advance!

@antoine-tran
Copy link
Contributor

antoine-tran commented Jan 31, 2024

Hi @emi-dm , we could not reproduce the errors (tried with linux and Mac). Could you paste the errors you got here ?

@Flux9665
Copy link

Flux9665 commented Feb 2, 2024

I believe audiocraft is missing from the dependencies, which is imported in audioseal.builder

Unfortunately, audiocraft itself comes with a ton of dependencies, which have their own issues, e.g. the required xFormers version requires PyTorch to be built with cuda, which doesn't work on inference machines without a GPU, but that is a different problem.

Another problem I encountered after installing audiocraft manually is that the URI in the example code from the readme cannot be found somehow. Here is the error message:
audioseal.loader.ModelLoadError: Path or uri audioseal_wm_16bits is unknown or does not exist

@Flux9665
Copy link

Flux9665 commented Feb 2, 2024

On Linux it seems to work just fine, on Windows however, I have these issues. I managed to resolve all of them with not too much trouble by hardcoding the path for the config in load_local_model_config to a local copy I made of the file and using the URL from the config as input to load_generator. Then I also had to set the environment variable USER to some dummy value, in order to satisfy OmegaConf.

So since this seems to be a WIndows only issue, I'm not sure if you intend to take care of it, but it would probably not be too difficult.

@antoine-tran
Copy link
Contributor

Thanks for the report, I could confirm I reproduced it in a non-Meta machine with no links to internal repos.

I fixed and closed the issue

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