vrm-c / vrm-specification

vrm specification
235 stars 37 forks source link

VRM1.0のschema "lookAt.rangeMap.schema.json" にて "inputMaxValue"と"outputScale"をrequiredとして宣言する #488

Closed tsgcpp closed 3 weeks ago

tsgcpp commented 1 month ago

提案込みで提出させていただきます。

lookAtの rangeMapHorizontalInner, rangeMapHorizontalOuter, rangeMapVerticalDown, RangeMapVerticalDown はそれぞれoptionalと思われますが、 非nullとして指定があった場合rangeMapの"inputMaxValue"と"outputScale"は両方必須パラメータ扱いが良いのではと考えております。

UniVRMのVrm10Importerでのロード処理においては上記扱いをしていそうと考えています。

もし"inputMaxValue"と"outputScale"を引き続き任意パラメータにするのであれば、defaultをschemaに記載する方がよさそうではないかとも考えております。

よろしくお願いします。

0b5vr commented 3 weeks ago

Related: #489

もともと必須パラメータでなかったプロパティを必須にするのは破壊的変更になるため、受容できません。

対応方針について、 #489 の方で議論させていただければと思います。