dmMaze / BallonsTranslator

深度学习辅助漫画翻译工具, 支持一键机翻和简单的图像/文本编辑 | Yet another computer-aided comic/manga translation tool powered by deeplearning
GNU General Public License v3.0
2.41k stars 162 forks source link

Update SakuraTranslator #434

Closed PiDanShouRouZhouXD closed 4 months ago

PiDanShouRouZhouXD commented 4 months ago

Update Sakura Translator to support SakuraDict and adjust the re-translation function for better handling of model degradation.

(另外,仍存在无法实时正确更新配置的问题)

dmMaze commented 4 months ago

具体是哪部分没法正确更新呢,看起来是 dict path 改完后需要重新加载字典

每次用户修改设置 updateParam 都会被调用,所以你可以参照下面 https://github.com/dmMaze/BallonsTranslator/blob/758a7fdcdc5fbce0fff365d9ca303958a7dad43f/modules/translators/trans_sugoi.py#L30-L35 改成

 def updateParam(self, param_key: str, param_content): 
     super().updateParam(param_key, param_content) 
     if param_key == 'dict path': 
         ...

另外请加下路径是否存在的判定,用try catch 把加载字典包一下

PiDanShouRouZhouXD commented 4 months ago

image 已经修复,另外,路径是否存在的判定是在字典类中进行的

dmMaze commented 4 months ago

其实调用完 super.updateParams 后 params 的对应内容就更新过了,不需要额外赋值

PiDanShouRouZhouXD commented 4 months ago

其实调用完 super.updateParams 后 params 的对应内容就更新过了,不需要额外赋值

可能是我哪里写错了,似乎去掉后面的内容后并不能成功更新,请问能帮忙看一下吗?

dmMaze commented 4 months ago

其实调用完 super.updateParams 后 params 的对应内容就更新过了,不需要额外赋值

可能是我哪里写错了,似乎去掉后面的内容后并不能成功更新,请问能帮忙看一下吗?

具体是哪部分内容不更新,我切你的 branch 后在 super.updateParams 后面打印 print(param_key, param_content, self.params) 在界面里改设置看到几个可选项都更新了啊

PiDanShouRouZhouXD commented 4 months ago

其实调用完 super.updateParams 后 params 的对应内容就更新过了,不需要额外赋值

可能是我哪里写错了,似乎去掉后面的内容后并不能成功更新,请问能帮忙看一下吗?

具体是哪部分内容不更新,我切你的 branch 后在 super.updateParams 后面打印 print(param_key, param_content, self.params) 在界面里改设置看到几个可选项都更新了啊

是字典无法正确更新。我知道为什么了,因为字典本身不是property,只有字典路径是,所以需要手动更新一下。

dmMaze commented 4 months ago

其实调用完 super.updateParams 后 params 的对应内容就更新过了,不需要额外赋值

可能是我哪里写错了,似乎去掉后面的内容后并不能成功更新,请问能帮忙看一下吗?

具体是哪部分内容不更新,我切你的 branch 后在 super.updateParams 后面打印 print(param_key, param_content, self.params) 在界面里改设置看到几个可选项都更新了啊

是字典无法正确更新。我知道为什么了,因为字典本身不是property,只有字典路径是,所以需要手动更新一下。

那其它的额外赋值就删掉吧...

PiDanShouRouZhouXD commented 4 months ago

看起来改 sakura 版本也要刷一下字典

已修复。