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

Fix bugs shape quantize, outs #175

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

Conversation

huutuongtu
Copy link

@huutuongtu huutuongtu commented Apr 1, 2024

Fix size for quantized_buf residual, prosody, phone.

✨ Description

Some audio will have bug with shape when I try voice conversion, so I changed some code to fix:

[1048] outs += out
RuntimeError: The size of tensor a (335) must match the size of tensor b (334) at non-singleton dimension 2

👨‍💻 Changes Proposed

  • Change shape residual, out, x, ...

🧑‍🤝‍🧑 Who Can Review?

@HeCheng0625

✅ 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)

fix size for quantized_buf residual, prosody, phone
@huutuongtu huutuongtu changed the title Update facodec.py Fix bug shape quantize Apr 1, 2024
@huutuongtu huutuongtu changed the title Fix bug shape quantize Fix bugs shape quantize, outs Apr 1, 2024
@RMSnow RMSnow requested a review from HeCheng0625 April 2, 2024 11:48
@yuantuo666
Copy link
Collaborator

@huutuongtu Hi, please install black and run it to reformat the code, ensuring the GitHub auto-check is successful.

@HarryHe11 HarryHe11 self-requested a review April 6, 2024 03:56
@HarryHe11
Copy link
Collaborator

@HeCheng0625 Hi, yuancheng, could you take a look at this pr about bugs in FaCodec when u r available? The codes look good to me.

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