chore!: v0.4.1をマージする

[sabonerune]

内容

r9y9/pyopenjtalkのv0.4.0をマージします。

関連 Issue

resolve #24

その他

ユーザー辞書のAPIはいったん本家の方に合わせました。
VOICEVOX Engineで関数名を変更する必要があります。

バージョン番号がgitの履歴とタグで自動決定されるようになったので本家のv0.4.0がある場所に0.4.0タグをつけるべきな気がします。
(先頭のvをつけるとpypiへリリースするCIが動いてしまうのでvはつけてはいけない)

前回マージされてから追加された主な機能

• ユーザー辞書
• ビルド周りの修正
• レガシーコードの修正
• GILのリリースとロックの追加

[sabonerune]

@sevenc-nanashi

• create_user_dict() set_user_dict()は削除
• unset_user_dict()は維持
• unset_user_dict()相当の機能を本家にPRを出し、マージされたらunset_user_dict()を削除。

という流れでしょうか?

多分それで問題ないと思います。

[sevenc-nanashi]

です。

[sabonerune]

unset_user_dict()を復活させました。

あとは本家へのPRを考えるだけ…
unset_user_dict()を他の関数と同じような名前に合わせるかupdate_global_jtalk_with_user_dict(path)でNoneを指定可能にするのどちらがいいと思いますか?

[sevenc-nanashi]

どっちでも大丈夫だと思います。
自分はupdate_global_jtalk_with_user_dict(None)が好きですね。（どっちで行くかはサボ寝ルネさんの好みにおまかせします）

[Hiroshiba]

LGTM！！

あ、warning出すのってどうなったんでしたっけ 👀

[Hiroshiba]

こちらどうやらuv移行の障壁になってるかもでした！！

• chore: uvへ移行する(Dockerコンテナ以外) voicevox_engine#1616 (comment)

今のこのPRのステートは･･･どこだろう、このPRの @sevenc-nanashi さんのapproveと、タスクのissue化？

[sevenc-nanashi]

unset_user_dict()を他の関数と同じような名前に合わせるかupdate_global_jtalk_with_user_dict(path)でNoneを指定可能にする

これをどっちにするのか待機してる感じです。

[Hiroshiba]

:naruhodo: です！！
マージ後に考えるとかissue建てる、という流れもあるかも（これはファシリテーター側の重要度・関心事・好みもありそう）

[sabonerune]

@sevenc-nanashi あ、すみません。
既に本家にPR出していました。

• ENH: Add support for unsetting a dictionary once set. r9y9/pyopenjtalk#105

[Hiroshiba]

本家もupdate_global_jtalk_with_user_dict(None)で統一する方針でPRが出ていて、現状の実装で問題なさそうに見えました！

nanashiさんのご判断で進めていただければ、お願いできれば･･･！

[sevenc-nanashi]

今の実装だとunset_user_dictになってそう？
次に追従するときにエンジン側をunset_user_dictから変更するでも大丈夫だと思います。
今のうちにupdate_global_jtalk_with_user_dict(None)にしちゃって次の追従を楽にするのもあり。
どうしましょう？

[Hiroshiba]

@sevenc-nanashi 特に強い希望がない限りこれに従うと良さそう？

[sevenc-nanashi]

それでよさそう。（unset_user_dictをupdate_global_jtalk_with_user_dict(None)に勝手に読みかえてました、申し訳ない...）

[Hiroshiba]

あ～
今確認したらunset_user_dictだけ前の API 名のままなんですねぇ。
まあでもプルリクエストがマージされるかわからないし、一旦このままでも良さそう！

ff mergeします！！

ちなみにこれ破壊的変更ですけど、issueとかREADMEの更新ってやるんでしたっけ 👀 @sevenc-nanashi

[sevenc-nanashi]

んー、pypiに上がってない = バージョンがほぼ無意味、かつボイボエンジンくらいしか使ってる人がいないと信じてなしでいいと思います。

[Hiroshiba]

なるほどです、結構同感です！
まあ･･･････ちょっとせっかくなんでissueだけ作って即クローズしますか！
ちょっと作ってみようと思います 🙏

[sabonerune]

gitのタグでバージョンを管理するようになったので 0f0fc44 に0.4.1タグをつけた方がいいと思います。
今pip install git+git+https://github.com/VOICEVOX/pyopenjtalk.gitでインストールするとバージョンが0.1.6.dev133+g74703b0になります。

先頭にvを付けるとpypiへのリリースを行うジョブが働いてしまうのでつけるべきではない

pyopenjtalk/.github/workflows/ci.yaml

Lines 54 to 57 in 74703b0

	build-for-publish:
	name: Build distribution 📦
	runs-on: ubuntu-latest
	if: "startsWith(github.ref, 'refs/tags')"

pyopenjtalk/.github/workflows/ci.yaml

Lines 83 to 86 in 74703b0

	publish-to-pypi:
	name: >-
	Publish Python 🐍 distribution 📦 to PyPI
	if: startsWith(github.ref, 'refs/tags/') # only publish to PyPI on tag pushes

[Hiroshiba]

なるほどです、忘れていました！！！

････････タグ全部消しますか！！！！！！！！！！！！
メンテナンスするの大変だし、このバージョンを必要としてる人いないだろうし、残ってるタグはフォークした時に偶然くっついていたものだけだし。

全部消したらどうなるかわからないけど、まあ今後ずっとバージョンに関して気にしなくていいので良さそう感！

[sabonerune]

確か0.1.devN+g....という感じのバージョンになると思います。
問題はパッと見て何のバージョンが入っているか分からないくらいで実害はなさそうですね。

[Hiroshiba]

ありがとうございます！！

ちなみにuvで
uv add git+https://github.com/VOICEVOX/pyopenjtalk@74703b034dd90a1f199f49bb70bf3b66b1728a86 --no-cache
としたらversionのとこが空白になってしまいました 😇

問題があるのかどうか分かりませんが、仕様を探しに行かないといけないフェーズがもう1回あるかもですね。。。