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 TransformerVC & VITSVC implementation #183

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

ArkhamImp
Copy link
Collaborator

@ArkhamImp ArkhamImp commented Apr 17, 2024

✨ Description

VC implementations with Transformer and VITS model architectures.
Samples refer to the previous PR: #90

πŸ‘¨β€πŸ’» Changes Proposed

  • Add Hubert token as input
  • Add speaker embedding as condition for zero-shot VC

πŸ§‘β€πŸ€β€πŸ§‘ Who Can Review?

@lmxue @RMSnow

βœ… Checklist

  • [βœ… ] Code has been reviewed
  • [βœ… ] Code complies with the project's code standards and best practices
  • [βœ… ] Code has passed all tests
  • [βœ… ] Code does not affect the normal use of existing features
  • [βœ… ] Code has been commented properly
  • [ βœ…] Documentation has been updated (if applicable)
  • Demo/checkpoint has been attached (if applicable)

@RMSnow
Copy link
Collaborator

RMSnow commented Apr 25, 2024

Hi @ArkhamImp, Haorui @HarryHe11 and I are also developing the zero-shot VC/SVC. We will leave this PR to review when we find a unified way to integrate all our code including yours!

@RMSnow RMSnow requested a review from HarryHe11 April 25, 2024 02:52
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.

3 participants