VOICEVOX / voicevox_core

無料で使える中品質なテキスト読み上げソフトウェア、VOICEVOXのコア
https://voicevox.hiroshiba.jp/
MIT License
866 stars 117 forks source link

バージョンを振る #677

Closed tuna2134 closed 11 months ago

tuna2134 commented 11 months ago

概要

現状Python向けのものは適当なバージョン振られていないため、今後のアップロードに影響があるため、振っていただけると助かります。

改善方法

python向けのライブラリのCargo.tomlのversionの部分を変更すればいいと思います。

sevenc-nanashi commented 11 months ago

https://github.com/VOICEVOX/voicevox_core/blob/main/.github/workflows/build_and_deploy.yml#L253 で振られてますね、タグとバージョンが一致しないミスとか考えるとこれがいいのかなぁと思います。

Hiroshiba commented 11 months ago

あ、バージョンはビルド時にGithub workflowから与えられる設計になっています! バージョンをコーディングすると、バージョンアップやテストビルドの度にコードを更新したりプルリクエストを送る必要があって不便だったので、今はそうなっています。

workflowのbuild_and_deploy.ymlのVERSION引数や環境変数を追うとどうなってるのかわかりやすいかなと…!

tuna2134 commented 11 months ago

となると、Py側の変更が良さそう?ですかね? ただ、tar.gzに収納される際、どうなるか気になりますですし voicevox.__version__でバージョンをpy側から呼び出さないといけないと思うので。。。

Hiroshiba commented 11 months ago

こちらのPyPIビルドの話ですよね。

コードに依りそうですが、同じくGithub workflowのworkflow_dispatchやworkflow_callで引数指定する形にしていただけると運用的には助かります。 Python APIは↓のようになってるので、参考になれば!

バージョン情報のset https://github.com/VOICEVOX/voicevox_core/blob/4c4b7677b3693ed3a361422c1c9819595270110c/.github/workflows/build_and_deploy.yml#L253

Python APIのビルド https://github.com/VOICEVOX/voicevox_core/blob/4c4b7677b3693ed3a361422c1c9819595270110c/.github/workflows/build_and_deploy.yml#L281

tuna2134 commented 11 months ago

voicevox.versionでバージョンをpy側から呼び出さないといけないと思うので。。。

これに関してどうしますか?

Hiroshiba commented 11 months ago

__version__は、ここかなと https://github.com/VOICEVOX/voicevox_core/blob/4c4b7677b3693ed3a361422c1c9819595270110c/crates/voicevox_core_python_api/src/lib.rs#L24

tuna2134 commented 11 months ago

__version__は、ここかなと

https://github.com/VOICEVOX/voicevox_core/blob/4c4b7677b3693ed3a361422c1c9819595270110c/crates/voicevox_core_python_api/src/lib.rs#L24

なら、なんとかなりそうですね、、、 クローズでよさそうかな?

Hiroshiba commented 11 months ago

ですね、一旦解決だと思います!