HuaHuaY commented 1 year ago

20 没有生效。看commit记录， 0dfc12a0110d4fe1b935aaafd1c20dcca387bead 中if恒为False，也没有相应的配置选项。

hsj1992 commented 1 year ago

有配置选项啊，是添加在utils.py里的，你看commit没看下里面的代码吗？因为提交者的考虑，所以默认是False没开启的。

要使用的话，在utils.py的131行，把 clean_ckpt = False 改为True 保存就启用了。在训练时执行到save_checkpoint时就会触发执行。

触发时的INFO log示例：

INFO:32k:Train Epoch: 46 [8%]
INFO:32k:[2.27864933013916, 2.4238224029541016, 10.692163467407227, 20.082901000976562, 1.0792361497879028, 11000, 9.942661422663591e-05]

INFO:32k:Saving model and optimizer state at iteration 46 to ./logs/32k/G_11000.pth

INFO:32k:.. Free up space by deleting ckpt logs/32k/G_8000.pth

INFO:32k:Saving model and optimizer state at iteration 46 to ./logs/32k/D_11000.pth

INFO:32k:.. Free up space by deleting ckpt logs/32k/D_8000.pth

INFO:32k:Train Epoch: 46 [90%]

SuCicada commented 1 year ago

看过去的issue描述应该是目前还没有添加删除开关，这部分代码应该还需要改进一下

HuaHuaY commented 1 year ago

呃，我想指的是preprocess_flist_config生成的config.json。直接硬编码的代码不能算配置选项吧。也没有额外的注释，不知道的以为是测试代码忘删干净了。

SuCicada commented 1 year ago

呃，我想指的是preprocess_flist_config生成的config.json。直接硬编码的代码不能算配置选项吧。也没有额外的注释，不知道的以为是测试代码忘删干净了。

是的，看起来这里代码没写完，过两天如果这里还没有改动我就提交一个pr吧

SuCicada commented 1 year ago

是的，看起来这里代码没写完，过两天如果这里还没有改动我就提交一个pr吧

已改动，已提交PR。https://github.com/innnky/so-vits-svc/pull/92

innnky / so-vits-svc

clean checkpoints没有生效？ #35

20 没有生效。看commit记录， 0dfc12a0110d4fe1b935aaafd1c20dcca387bead 中if恒为False，也没有相应的配置选项。