Skip to content

chore: downloaded_version!の利用をやめる#1200

Merged
qryxip merged 2 commits intoVOICEVOX:mainfrom
qryxip:pr/chore-stop-using-downloaded-version-macro
Nov 15, 2025
Merged

chore: downloaded_version!の利用をやめる#1200
qryxip merged 2 commits intoVOICEVOX:mainfrom
qryxip:pr/chore-stop-using-downloaded-version-macro

Conversation

@qryxip
Copy link
Member

@qryxip qryxip commented Nov 14, 2025

内容

目的は二つ。一つは VOICEVOX/ort#17 の準備で、もう一つは #1199 の解決。

特に後者については、 #1198 の試みにすら牙を剥いてきているため先に解消しておきたい。

関連 Issue

Fixes: #1192
Fixes: #1199

その他

@qryxip qryxip requested a review from Hiroshiba November 14, 2025 23:05
@qryxip qryxip force-pushed the pr/chore-stop-using-downloaded-version-macro branch from 38dfbde to 58c9da5 Compare November 14, 2025 23:10
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 14, 2025

CodSpeed Performance Report

Merging #1200 will not alter performance

Comparing qryxip:pr/chore-stop-using-downloaded-version-macro (63a1b4a) with main (b4766c4)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 2 untouched
⏩ 22 skipped1

Footnotes

  1. 22 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

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!

なんでこれで解決するのかわからなかったのでPR内にコメント書き残しとくと良さそう。
あとPRタイトルはort::downloaded_version!の利用をとしとくと後で見たときによりわかりやすそう。

@@ -0,0 +1 @@
1.17.3 No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

これは何の目的でいるのか、運用方法どうするかなんとなくで良いのでメモしておきたいかも。
なんでこれがあるのか全くわからなそう。

docsやどこかのreadmeに残すのが良いけど、このPRにコメント残すとかでも別に良さそう。

Copy link
Member Author

Choose a reason for hiding this comment

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

あえて言語化するならOnnxruntime::LIB_VERSIONの設定とか、test_utilから純正ONNX RuntimeのDLLのファイル名を把握するためとか、将来的にvoicevox-ortからではなくこのリポジトリの責務で純正ONNX Runtimeのダウンロードを行うためとかですが、正直grepすれば一発なのでファイルにはあまり書きたい気持ちが無いです。「このPRにコメント残す」の方で。

@qryxip
Copy link
Member Author

qryxip commented Nov 15, 2025

あとPRタイトルはort::downloaded_version!の利用をとしとくと後で見たときによりわかりやすそう。

downloaded_version!はpykeio/ortにはなくてvoicevox-ortにのみあるものなので、一応ですが混乱を避けるためでした。

@qryxip
Copy link
Member Author

qryxip commented Nov 15, 2025

何故かはわからないが #1192 の症状が収まった。#1192の本質はもしかしたら #1199 だったのかもしれない。とりあえず#1192もリンク。

@qryxip qryxip merged commit 5d6c188 into VOICEVOX:main Nov 15, 2025
36 checks passed
@qryxip qryxip deleted the pr/chore-stop-using-downloaded-version-macro branch November 15, 2025 05:36
qryxip added a commit to VOICEVOX/ort that referenced this pull request Jan 12, 2026
VOICEVOX/voicevox_core#1200 により不要になった`downloaded_version!`を削
除する。

#17 の一環。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants