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

Inference_coqui tts pre-trained models #1

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Inference_coqui tts pre-trained models #1

wants to merge 9 commits into from

Conversation

Nikav7
Copy link
Owner

@Nikav7 Nikav7 commented Nov 15, 2022

@s1nayak @FeNlay

@Nikav7
Copy link
Owner Author

Nikav7 commented Nov 15, 2022

@ZhengkunMei

@Nikav7
Copy link
Owner Author

Nikav7 commented Nov 16, 2022

@FenNlay

@FenNlay
Copy link

FenNlay commented Nov 21, 2022

I tried the notebooks. In the READ.me it is stated that there a three notebooks. However, only two are listed. The second notebook listed isn't one that could be found in the file-list, but I guessed it was probably supposed to be Assignment7_Veronica_.ipynb.

For the READ.me more information could be given for the two notebooks as indicated by the assignment/rubric. For example what one can do in the notebook and what the output would be.

Notebook 1 was very entertaining. The instructions were clear. It would be nice to have an audioplayer in the notebook after the generated audio, so it could be played easier.

Notebook 2 also had straightforward instructions. Sadly it didn't work were it was already mentioned to cause an error, even after trying the suggested solution. This seemed to be known by the author given the statement in the last code snippet.

Suggested grade: 7/10

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