-
Notifications
You must be signed in to change notification settings - Fork 205
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
モーフィング関係で、スタイルIDなのにspeakerになっているところをstyle_idに変更 #826
The head ref may contain hidden characters: "\u30E2\u30FC\u30D5\u30A3\u30F3\u30B0\u95A2\u4FC2\u3067\u3001\u30B9\u30BF\u30A4\u30EBID\u306A\u306E\u306Bspeaker\u306B\u306A\u3063\u3066\u3044\u308B\u3068\u3053\u308D\u3092style_id\u306B\u5909\u66F4"
モーフィング関係で、スタイルIDなのにspeakerになっているところをstyle_idに変更 #826
Conversation
…のにspeakerになっているところをstyle_idに変更
ちょっと自信無いですが、StyleId化辺りで使うのでマージしたいと思います。 |
コンフリクト解消したのでマージしようとしたけど、mypyの追加でエラーになっていました。。一旦draftにします。。 📝 Optionalな型の引数と同じ変数名にOptionalじゃない値を再代入しようとすると怒られるので、引数名の方を |
…いるところをstyle_idに変更
…のにspeakerになっているところをstyle_idに変更
e78f1c0
to
358d494
Compare
StyleId周りが解決できたのでdraft開けました!! |
たぶん大丈夫だと思うのでマージします・・・! |
内容
のとどめプルリクエストです。
モーフィング周り以外はこちらのプルリクエストで完了済みです。
関連 Issue
close #589
その他
ドキュメント周りも一通り見直して全部変えたつもりです。
またこちらのPRでできていなかったStyleId化も一通り変更できたと思います。
(
[int
辺りで全体検索してDict[int
とかList[int
とかの漏れがなさそうなことをチェックしました。あとはユーザー辞書の単語priorityとかだけなはず。)