TextAnalyzer traitにstring->AccentPhraseModel[]を移動

[eyr1n]

内容

#730 で話されている，TextAnalyzer interfaceを実装するための準備として，Rust側にTextAnalyzer traitを生やしました．

また，

• AquesTalk風記法からAccentPhraseの配列を生成するTextAnalyzer
• OpenJtalkからAccentPhraseの配列を生成するTextAnalyzer

を実装するためにSynthesizerからコードを移動し，これら2つのTextAnalyzerをSynthesizerに持たせました．

Issue全体の実装をする(_kana系APIとOpenJTalkの統合)を行うと，Synthesizerを触る部分全体にAPI変更が波及してしまうため，外から見たAPIの変更は現時点では全く行っていません．

関連 Issue

#730

その他

[eyr1n]

現時点で懸念しているのが，果たして一つのSynthesizerインスタンスに対してKanaAnalyzerとOpenJTalkAnalyzerが排他でいいのか？ということです．

今の段階ではそもそもSyhtnesizer内に二つのTextAnalyzerを持たせているので問題ありませんが，今後実装するであろうSynthesizer<OpenJTalkAnalyzer>によって生成されたインスタンスでAquesTalk風記法(..._kana系API)が使えなくても良いのかどうかが疑問です．

[eyr1n]

pub struct Synthesizer<T: TextAnalyzer> {
    pub(super) status: Status<InferenceRuntimeImpl, InferenceDomainImpl>,
    text_analyzer: T,
    use_gpu: bool,
}

上記の形での実装もすでに行っていますが，#740 (comment) について検討してから別でPRを出すつもりです．

[qryxip]

stateをArcに包んでSynthesizerをclonableにし、Synthesizer::with_text_analyzer: fn(&self, T2) -> Synthesizer<T2>みたいなのを用意すればstatusを共有したまま併用できるかと思います。

[eyr1n]

返信ありがとうございます。
たしかに共有してしまえば良さそうですね、APIに変更が生じるのでまた別のPRとして現在作業中のものを出してみます。

[qryxip]

LGTM!

CC: @femshima, @cm-ayf

[Hiroshiba]

LGTM！！！