-
Notifications
You must be signed in to change notification settings - Fork 117
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
change: InferenceFailed
→ MlInference
#821
change: InferenceFailed
→ MlInference
#821
Conversation
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.
そこまで気にしてないのですが、Ml
に関して、省略語は他の勘違いをして間違って伝わってしまう可能性があるので避けたほうが良いかもです。
(Clean Code本とかで言われている点)
他の部分でMl
というプレフィックスが大量に使われてたら、「なんか共通してついてる接頭語」という観点でありかもですが、現状そうでもなさそうなので。
ちなみにMLは機械学習以外に機械言語(Machine Language)、メーリングリスト、ミリリットルなどがあって意外と誤解しやすいかもです。
流石にミリリットルは無いだろうけど。
自然言語の"ML inference"というフレーズであると考えると、ほぼ"machine learning"に限られるのではないかと思っています。ChatGPT 4oに聞いてもこんな感じでした。 |
機械学習分野における推論という言葉を知っているAIに聞いたら、それはさすがに意味を分かってくれると思います。 ちなみに
|
理由としては"inference"自体は広範な用法がある一方"ML inference"ならほぼ一意になり、また |
なるほどです。 onnxがMLにより作られたものであるということは、コアのコード内やAPIの意識から外すべきかもです。(理由は後述) あと、 機械学習という言葉を避けたい理由単純にコアの本質に関係ないはずだからです。 例えば平均と分散を変えるだけの計算処理をするonnxも作れるのですが、これはもはや機械学習じゃないんですよね。 この辺が機械学習という名前を避けたい理由です。 |
なるほど。モック以外でもあり得る感じですかね?
単純に |
少なくともVOICEVOX製品版ではほぼありえない、くらいの温度感ではあります!
あーーーーーーーーーーー良いと思います!!!!! |
内容
InferenceFailed{Error,Exception}
をMlInference{Error,Exception}
にします。関連 Issue
Resolves #779.
その他