Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Bug fix for VoxtLM v1 recipe#5782

Merged
sw005320 merged 11 commits intoespnet:masterfrom
cromz22:voxtlm_v1
May 20, 2024
Merged

Bug fix for VoxtLM v1 recipe#5782
sw005320 merged 11 commits intoespnet:masterfrom
cromz22:voxtlm_v1

Conversation

@cromz22
Copy link
Contributor

@cromz22 cromz22 commented May 20, 2024

What?

Fixed bugs when running egs2/voxtlm_v1/lm1/run.sh. Specifically:

Notes

  • Here I kept the changes minimal so that current recipe runs smoothly. More changes could be made to improve the code.
    • Especially I think it would be better to include TTS unit-to-wav generation stage in the recipe. Help from the original authors is appreciated.
  • I ran the code with reduced data to speed up the debugging process. I did not check the performance of full training.

Why?

The recipe did not run smoothly as it is. See the issue below for more detail.

See also

@sw005320 sw005320 added the LM label May 20, 2024
@sw005320 sw005320 added this to the v.202405 milestone May 20, 2024
@sw005320
Copy link
Contributor

Thanks, @cromz22!
This is really helpful.
After the CI check, I'll merge this PR.

@cromz22
Copy link
Contributor Author

cromz22 commented May 20, 2024

Looks like it has passed the tests other than the MacOS installation issue.

@sw005320 sw005320 merged commit a83f82b into espnet:master May 20, 2024
@sw005320
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants