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.

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

asyncにできるところをasyncにして高速化する #1071

Closed
3 tasks
sevenc-nanashi opened this issue Feb 25, 2024 · 6 comments
Closed
3 tasks

asyncにできるところをasyncにして高速化する #1071

sevenc-nanashi opened this issue Feb 25, 2024 · 6 comments

Comments

@sevenc-nanashi
Copy link
Member

内容

タイトル通りです。具体的にはGET /speaker_infoあたりが該当すると思います。

Pros 良くなる点

  • 速くなる

Cons 悪くなる点

  • ないはず?

実現方法

aiofilesとかを使う?

VOICEVOXのバージョン

0.?.0

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

  • Windows
  • macOS
  • Linux

その他

(なし)

@tarepan tarepan added the 要議論 実行する前に議論が必要そうなもの label Mar 5, 2024
@tarepan
Copy link
Contributor

tarepan commented Mar 8, 2024

#1073 でのチャレンジにより、定量指標を用いた高速化評価が必要だと明確化されました。
本 issue の方向性としては「asyncにできるところをasyncにして高速化」の前段階として「async の価値を測るための速度計測実装」が入りそうですね。

@tarepan tarepan self-assigned this Mar 8, 2024
@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 18, 2024

高速化する前にまず測定という鉄則をもとにすれば、たしかにまずは計測可能にするのが大切そうかもですね!
まあ知識があればわりと測定せずとも良いのですが、Pythonのasyncに関する知識がおそらく誰にもないという感じなので…。

@tarepan

This comment was marked as off-topic.

@tarepan tarepan added 状態:実装者募集 実装者を募集している状態 and removed 要議論 実行する前に議論が必要そうなもの labels Mar 18, 2024
@tarepan tarepan removed their assignment Mar 18, 2024
@Hiroshiba

This comment was marked as off-topic.

@tarepan

This comment was marked as off-topic.

@tarepan
Copy link
Contributor

tarepan commented Apr 2, 2024

#1134 により、無条件で async 化できる関数は全て async 化されました。
コレ以外の関数はそれぞれ個別の事情に合わせて高速化され、async 化はその中で取りうる 1 方策という位置づけになります。

よって async 化に主眼を置いた本 issue は resolved により close が妥当そうです。
async に主眼を置いて更なる改善が出来そうな場合は re-open して議論深められればと思います。
また個別関数の更なる改善もドシドシ募集しています、興味のある方は気軽に issue open してください👍️

@tarepan tarepan closed this as completed Apr 2, 2024
@tarepan tarepan removed the 状態:実装者募集 実装者を募集している状態 label Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants