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

Java APIに非同期版を追加する #769

Open
3 tasks done
qryxip opened this issue Mar 31, 2024 · 2 comments
Open
3 tasks done

Java APIに非同期版を追加する #769

qryxip opened this issue Mar 31, 2024 · 2 comments
Labels
OS：linux OS：mac OS：win 機能向上

Comments

@qryxip
Copy link
Member

qryxip commented Mar 31, 2024

内容

Apache OpenDALを参考に、CompletableFuture<T>で結果を返すAPIをJava APIに増やします。
#550 (comment)

Pros 良くなる点

  • Javaでも非同期機能が使えるようになる

Cons 悪くなる点

  • 実装、メンテ

実現方法

VOICEVOXのバージョン

N/A

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

OpenDALのJavaの非同期は、なんかちょっと変わる予定らしいです。
apache/opendal#4376
@qryxip qryxip changed the title Java APIに非同期機能を追加する Java APIに非同期版を追加する Mar 31, 2024
@Hiroshiba
Copy link
Member

実装しても良いような気はしつつ、需要が発生してからでも良いような気もしつつ、という直感です！
優先度は高くないけどプルリク来たらマージしても良いかも、くらいの温度感かな…？
（需要がわからないのがとても大きい）

@qryxip
Copy link
Member Author

qryxip commented Apr 10, 2024

まあ今入れるかどうかはともかく、名前空間は他言語と合わせる形でpackage blockingを切った方が多分よいですね。

@qryxip qryxip mentioned this issue Oct 23, 2024
Merged
qryxip added a commit that referenced this issue Nov 4, 2024
@qryxip
change!(java): blockingパッケージを作ってクラス移動 (#861)
006b4f0 
Java APIに`voicevoxcore.blocking`パッケージを作り、`Synthesizer`などをそ
こに入れる。それに伴い、同期版APIでも非同期版APIでも共通するであろう次の
クラスは分離して`blocking`の外のパッケージに置く。

- `voicevoxcore.{Synthesizer. => }AccelerationMode`
- `voicevoxcore.{VoiceModelFile. => }SpeakerMeta`
- `voicevoxcore.{VoiceModelFile. => }StyleMeta`
- `voicevoxcore.{VoiceModelFile. => }StyleType`
- `voicevoxcore.{UserDict.Word => UserDictWord}`

Refs: #769 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
OS：linux OS：mac OS：win 機能向上
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Hiroshiba @qryxip