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

Remove: OjtPhoneme.convert() の廃止 #786

Closed
3 tasks done
tarepan opened this issue Nov 27, 2023 · 1 comment · Fixed by #791
Closed
3 tasks done

Remove: OjtPhoneme.convert() の廃止 #786

tarepan opened this issue Nov 27, 2023 · 1 comment · Fixed by #791

Comments

@tarepan
Copy link
Contributor

tarepan commented Nov 27, 2023

内容

要望: OjtPhoneme.convert() の廃止

OjtPhoneme.convert() は音素表現の変換(silpau)を担う関数である。
__init__() ではなく関数として用意されており、任意タイミングの変換が想定されている。
しかし現在の実装では OjtPhoneme がインスタンス化される唯一の箇所 の直後で .convert() がコールされている。
すなわち現在のユースケースであれば __init__() 内で処理できる。

また現在の _synthesis_implpau 前提のコードであるため、sil にしておけるメリットは無いにも関わらず変換し忘れるデメリットのみが存在している。

上記の理由より、OjtPhoneme.convert() の廃止を提案します。

Pros 良くなる点

  • コード見通しの改善
  • 将来的バグ埋め込みの回避

Cons 悪くなる点

  • (現在未利用の)柔軟性低下

実現方法

  • __init__ での sil-to-pau 変換実装
  • .convert() の廃止

VOICEVOXのバージョン

0.14.10

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux
@tarepan tarepan changed the title Remove: OjtPhoneme.convert()の廃止 Remove: OjtPhoneme.convert() の廃止 Nov 27, 2023
@Hiroshiba
Copy link
Member

いいですね、消しちゃいましょう!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants