vrm-c / vrm.dev

Home Page:https://vrm.dev/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Human bone spec was changed silently.

saturday06 opened this issue · comments

https://github.com/vrm-c/vrm.dev/pull/29/files こちらのマージ済みのプルリクエストですが、 以前まで [^notrequired] の記載が無かった chestneck のボーンが Optional とマークされていて仕様変更が発生しています。そのため、spec versionを増やすか、この変更部分だけリバートをする必要があります。

関連する議論はこちらで行われています→ vrm-c/vrm-specification#10

ただ、現状UnityのUIが chestneck をOptionalとしていることから現状の仕様はユーザーに優しくないので、取り急ぎspec versionを0.2へ更新し、そこで chestneck をOptionalボーンとしてしまっても良いのかなと個人的に思っています。

以前必須ボーンについて ousttrue さんに伺った際の議論はこちら→ https://github.com/vrm-c/UniVRM/issues/113#issuecomment-451852579

commented

ご報告ありがとうございます。
実装(Unity)と一致させたほうが分かりやすいのは確かにそうですね。
いったんchestとneckは元の状態(必須)に戻し、議論の結果に合わせて変更することに致します。

変更確認したためクローズします。ありがとうございました。