-
Notifications
You must be signed in to change notification settings - Fork 119
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
Add iOS support #471
Add iOS support #471
Conversation
target : - x86_64-apple-ios - aarch64-apple-ios - aarch64-apple-ios-sim
about process_path for iOS support Co-authored-by: Ryo Yamashita <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! これでコアの iOS ビルドが(おそらく)完成ですね。 @HyodaKazuaki さんありがとうございました……!
@Hiroshiba ちょっと考えていることを書きます。おそらく iOS 版コアを使いやすいようにするために、少し追加の作業する必要があるかもしれないと考えています。 #459 (comment) や #459 (comment) に書かれている話と関連するのですが、実機向けのコアとシミュレータ向けのコアを framework(さらにはそれらをまとめる xcframework)にまとめて zip で固め、配布用のリポジトリを作ってそこで公開し、Cocoapods で導入できるようにすると、voicevox_mobile の開発向けにも使いやすくなる可能性があります。
この辺りはまだ調査中なのですが、遠くないうちに Issue を立てようと思います(なにぶん詳しくない分野なので、Issue での議論の中で情報を集めることになるかもです。 https://moneyforward-dev.jp/entry/2021/11/11/private-cocoapods-library/ などには参考になる情報が書いてありました。こちらに書いてあるのは Swift 製ライブラリの Pods についてですが、公式配布されている onnxruntime が C/C++ ライブラリの xcframework の Pods であることを考えると、組み合わせるとうまくいくのではと推察しています)。
この点については私も調べていたのですが、未だ有益な情報を得られていません。 |
@PickledChair なるほどです、面白そうです!! あ、配布用のリポジトリは別に作っても、このリポジトリに作っちゃっても良いかもです。 どちらでも、やりやすい方で……! |
内容
iOSの各アーキテクチャに対応します。
対応アーキテクチャ
関連 Issue
close #459
その他
process_pathについては、 wesleywiser/process_path#16 が残念ながらMergeされる見込みが立たなかったため、 VOICEVOX/process_path へ移行しました。