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.

Sign up for GitHub

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

Jump to bottom

Merge 0.15.5 #894

Merged
merged 11 commits into from
Dec 18, 2024
Merged

Merge 0.15.5 #894

merged 11 commits into from
Dec 18, 2024

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Dec 14, 2024

内容

詳細はマージコミットのコミットメッセージに書きました。このコミットメッセージもレビュー対象にして頂きたく。

関連 Issue

Resolves: #737

その他

参考: 0.14.6...0.15.5

Hiroshiba and others added 9 commits January 27, 2024 17:02
@Hiroshiba
[project-s] ピッチ輪郭推論を追加 (VOICEVOX#531)
c7dfb03
@y-chan @Hiroshiba
[project-s] ハミング機能・歌機能向けのモデル・API(compatible_engineのみ)を追加 (VOICEVOX#724)
96bd927 
* remove contour and rename to talk xxx

* fix speaker id map

* rename functions and variables

* add models to model file

* add sing style and source filter models to model file set

* add new models to status

* rename get model index and speaker id

* add new models session

* change i32 to i64

* add new predictor to inference core

* add new predictor to core

* add new predictor to compatible engine

* rename source filter to sf decode

* fix rename miss

* rename sing style to sing teacher

* fix rename miss

* remove vector

* add TODO comment (add sing tests)

Co-authored-by: Hiroshiba <[email protected]>

* fix comment out

* lint

---------

Co-authored-by: Hiroshiba <[email protected]>
@y-chan @Hiroshiba
fix is model loaded bug (VOICEVOX#729)
2f52377 
* fix is model loaded bug

* Apply suggestions from code review

---------

Co-authored-by: Hiroshiba <[email protected]>
@y-chan @Hiroshiba
[project-s] モデルへの入力の形・データを修正 (VOICEVOX#732)
fc91b55 
* update shape

* using f0

* fmt
@Hiroshiba
[project-s] project-s用のテストを作成（モデルがないのでコメントアウト中） (VOICEVOX#734)
01c3a9b 
project-s用のテストの枠組みだけ作った
@y-chan @Hiroshiba
[project-s] ダミーモデルの追加とテストの有効化 (VOICEVOX#736)
ca585b2 
* add models

* remove comment out

* fix merge miss

* add tests to status

* add onnx file to typos exclude files

* rename model file

* fmt

---------

Co-authored-by: Hiroshiba <[email protected]>
@Hiroshiba
[project-s] スタイルタイプの名称変更 (VOICEVOX#738)
0fd77a2 
* style_typeの名称変更

* いらない変更が入ってた
@qryxip
[release-0.15] .onnx/.binファイルの読み込みを遅延させる (VOICEVOX#768)
fe8358b
@Hiroshiba
[release-0.15] macos-11をmacos-12にアップデートしておく (VOICEVOX#813)
87df545 
* release-0.15のmacos-11をmacos-12にアップデートしておく

* 空コミット

* --locked

* 戻した

* 要らない変更
@qryxip qryxip requested a review from Hiroshiba December 14, 2024 14:53
@qryxip
Copy link
Member Author

qryxip commented Dec 14, 2024

テストはhttps://github.com/VOICEVOX/voicevox_core/tree/release-0.15にあるものしか入れておらず、動作はまだ未確認。

@qryxip qryxip mentioned this pull request Dec 14, 2024
Merged
@qryxip qryxip mentioned this pull request Dec 15, 2024
Merged
Hiroshiba
Hiroshiba approved these changes Dec 15, 2024
View reviewed changes
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM！！！

とりあえず一通り見てみました、問題なさそう！！
マージありがとうございます！！！！

@qryxip
chore: [0.15] remove obsolete parts (VOICEVOX#896)
0864f10 
以下のものを削除する。

- もう必要のないワークアラウンドである、cargo-binstallへの`--locked`
- デッドコードである、`UnsupportedModel`と
  `VOICEVOX_RESULT_UNSUPPORTED_MODEL_ERROR`
- 嘘コメントと化している、"TODO: sing系のテストを足す"

Refs: VOICEVOX#894
@qryxip
Merge commit '0864f106085e91a90ca60241f1f375db4ea4dfcc' into HEAD
a2452a1 
以下のようにしてマージ。

- ソングのモデルもsample.vvmの中に入れて一つにする。それに伴い、
  `release-0.15`で増えた一部のテストは削除。
- モデルのロードに関しては、現行のVVMを前提にした処理に置き換える。
- 推論の実行に関しては`ndarray`を前提とした処理に置き換える。
@qryxip
Copy link
Member Author

qryxip commented Dec 18, 2024 •
edited
Loading

@Hiroshiba #895 と #896 を含めました。
https://github.com/qryxip/voicevox_core/compare/1c343448b22528e0fb1005f5f4e0f7a9a86af77c..merge-0.15.5

ご確認願います

、と言いたいですがcbindgenの更新忘れがあったっぽい。けどなんかGitHubが不調っぽい。CIも落ちてる。
push通った。

@qryxip
Copy link
Member Author

qryxip commented Dec 18, 2024 •
edited
Loading

pushは通ったけどどうやらたまたま一時的に通っただけのようで、まだちょくちょく駄目っぽくてGHAが落ちる。
(GitHub Statusは現段階では全部 ✔️ だけど)

@qryxip
Copy link
Member Author

qryxip commented Dec 18, 2024

@Hiroshiba あ、あとCreate a merge commitを出来るようにして頂ければ。

Hiroshiba
Hiroshiba approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM！！

ff mergeも可能にします！
こうするとデフォルトのボタンの動作がff mergeしなることもあるので注意です！

@Hiroshiba
Copy link
Member

Hiroshiba commented Dec 18, 2024 •
edited
Loading

今気づいたのですが、こちらの方針とズレてるかも…？

ソング系API
compatible engine含め、0.16に含めない

記憶が曖昧ですが、たしか後ろ倒しになると新機能を追加してもリリース遅れるから、リリースをとにかく急ごうって感じでしたっけ。
提案ですが、このprはせっかくなのでマージしてあとは強い意志を持ってなるべく何も足さずにリリースまで駆け抜けるってのはどうでしょう？

@qryxip qryxip merged commit 1acedaf into VOICEVOX:main Dec 18, 2024
29 checks passed
@qryxip
Copy link
Member Author

qryxip commented Dec 18, 2024

#825 での心配事を潰しておきたいというのが目的でした。どちらかというと調査で、いけそうだったので一気にやった次第です。使った時間は多分5時間程度？

#545 に書きましたが、ちょっと他にもやっておきたいことはあります。APIの破壊的変更にはなるので…

@qryxip
Copy link
Member Author

qryxip commented Dec 18, 2024

あ、Create a merge commitはもう無効化してよいはず。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

project-s ブランチをmainブランチにマージしたい
3 participants
@qryxip @Hiroshiba @y-chan