-
Notifications
You must be signed in to change notification settings - Fork 206
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
2話者からモーフィングさせた音声を生成するAPIを追加 #151
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
素晴らしい!! 問題がなさそうな実装でした!!
キャッシュですが、確かにworldの解析結果をキャッシュする必要があって、確かに容量がなかなか大きそうな感じがしますね…
まあモーフィング用にほしいキャッシュは、おそらく直前で試している数個の音声だけだと思うので、32ほどキャッシュすれば十分な気がします。
32個くらいならキャッシュしてもそんなに大きいメモリにはならないはず?
キャッシュ判定は、ソフトウェアの方でもやっている、AudioQueryをjson化してなんかしらのハッシュ値を取る方法が使えるかもです。
あとはちょっとpythonの知識が必要ですが、lru_cacheというものが便利かもしれません。
わからなければ聞いていただければ…!
情報ありがとうございます! |
思った以上にメモリ消費が激しそうですね・・・ |
Coverage Result |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
本当に非の打ち所がない素晴らしいコードだと思いました!!
聞きたいことと、そこまで強い意見ではないけど変更をお願いしたいところがあったのでRequest changesにしています!
コードは完璧だと思います! あとせっかくなので利用方法の具体例をREADMEで紹介するのはどうでしょう?
windowsからcurlで日本語文字をPOSTするのは結構くせがあります。わからないところがあれば聞いていただければ・・・! |
確かにぱっと見使い方がわからないAPIではあるので、サンプルがあった方が良いですね... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! とても良いREADMEだと思います!
少し長い議論になりましたがとても良い機能になったと思います! お疲れさまでした!
一部細かい修正をこちらで加えちゃってマージしたいと思います。
(最終commitは僕になってしまいますが、貢献者一覧にはもちろん掲載させていただきます!)
内容
2つの話者で生成した音声をWORLDでモーフィングさせ、その結果を出力するAPIを追加します
関連 Issue
下記tweetのスレッドを参照
https://twitter.com/mes_deleted_n/status/1451845426533990407
その他
サンプルリクエスト
TODO