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

Add Generate Script #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add Generate Script #29

wants to merge 2 commits into from

Conversation

robdmac
Copy link

@robdmac robdmac commented Mar 5, 2019

This wraps the audio generation into a script and generates an audio file reminiscent of (and to make comparisons with) the tensorflow-wavenet repo: https://github.com/ibab/tensorflow-wavenet/blob/master/generate.py

robdmac added 2 commits March 4, 2019 21:39
This wraps the audio generation into a script and generates an audio file reminiscent of (and to make comparisons with) the tensorflow-wavenet repo.
@chrisdonahue
Copy link
Owner

Thanks for the PR and sorry it has taken me a while to get back to you. I like the idea of having this generate script, but it would make sense to also have one for SpecGAN. Can you possibly rename this to wavegan_generate.py and add a corresponding one for specgan? If that is too much effort, I can leave this PR open and get around to writing the code.

Also would be nice to add an argument specifying the number of samples to generate.

@itsKaspar
Copy link

itsKaspar commented Jun 28, 2019

ej, i'm having issues with the generating script
i've installed wavenet through pip and pip3
but when I run the script through P3 I get

ImportError: cannot import name 'WaveNetModel'

so I suppose the code isn't for P3
and when I run it through P2

ImportError: cannot import name ABC

and I can't seem to find a way to install abc
which package is this refering to ?

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.

None yet

3 participants