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

modify freeze_quantizer mode, avoid quantizer's codebook updating #953

Merged
merged 1 commit into from
May 2, 2024

Conversation

hcwu1993
Copy link
Contributor

@hcwu1993 hcwu1993 commented Apr 9, 2024

No description provided.

@hcwu1993
Copy link
Contributor Author

hcwu1993 commented Apr 9, 2024

i found when i set freeze_quantizer=true, the quantizer's codebook still updated. In that case, the VITS's codebook is different from the pretrained model. I think this may lead mismatch for the GPT between VITS. So i modify some code to avoid this case.

@RVC-Boss RVC-Boss merged commit a95e6c1 into RVC-Boss:main May 2, 2024
XXXXRT666 pushed a commit to XXXXRT666/GPT-SoVITS that referenced this pull request May 26, 2024
XXXXRT666 pushed a commit to XXXXRT666/GPT-SoVITS that referenced this pull request May 27, 2024
RVC-Boss pushed a commit that referenced this pull request May 27, 2024
* api接口,修复文本切分符号设定中,中文分号错写为英文分号的问题 (#1001)

* 一些小问题修复 (#1021)

* fix import error. It may happen when calling
 api.py

* Update README.md

* Update gpt-sovits_kaggle.ipynb

* Update gpt-sovits_kaggle.ipynb

* fix path error
delete useless line wraps

* 删除重复的 COPY 指令 (#1073)

* [优化] 1Aa-文本获取 (#1102)

* Filter unsupported languages

* add feedback

* simplify modification

* fix detail

* Update english.py (#1106)

copy but not ref the phones list becoz it will be extend later, if not do so,it will affect the self.cmu dict values.

* Update models.py

* modify freeze_quantizer mode, avoid quantizer's codebook updating (#953)

---------

Co-authored-by: FengQingYunDan <[email protected]>
Co-authored-by: Kenn Zhang <[email protected]>
Co-authored-by: 蓝梦实 <[email protected]>
Co-authored-by: lyris <[email protected]>
Co-authored-by: hcwu1993 <[email protected]>
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