Fix spec for fields which have indices, add minimum=0 for them

[yutopp]

Resolve #91 (partially).

Changes are below.

• vrm.firstperson.firstPersonBone
  • Add minimum=0
  • Now ignore a field when -1 is specified when exporting
• vrm.firstperson.meshannotation.mesh
  • Add minimum=0
• vrm.meta.texture
  • Add minimum=0
  • Now ignore a field when -1 is specified when exporting
• vrm.humanoid.bone.node
  • Add minimum=0
• vrm.secondaryanimation.spring.center
  • DO NOTHING for compatibility
• vrm.secondaryanimation.spring.bones
  • Add minimum=0 for elements
• vrm.secondaryanimation.spring.colliderGroups
  • Add minimum=0 for elements
• vrm.secondaryanimation.collidergroup.node
  • Add minimum=0 for elements

I have checked that VRM files exported with this version can be read with UniVRM 0.46c.

NOTE: vrm.secondaryanimation.spring.center cannot be changed because of compatibility problems.

[saturday06]

これを入れると、仕様の後方互換性が無くなりますが問題ないでしょうか？

個人的には、後方互換性の無い仕様変更は1.0まで延期お願いしたいです

[saturday06]

補足ですが、上記で言及している「仕様の後方互換性」とは「UniVRMの仕様の後方互換性」ではなく「json schemaとして記述されているVRM Version 0.0仕様の後方互換性」のことです。

[saturday06]

すみません、上記は早とちりでした。VRM 0.0の仕様として提供されているjson schemaへの変更であると勘違いしていました。UniVRMコード内部で使われるチェック機構に対する変更であることを確認しました。それならば問題無いかと思います。失礼しました 🙇

[yutopp]

おお、ありがとうございます。

おっしゃる通りで、実装上の制約の変更のみで、json schemaへの変更は含まれていません。また、UniVRMにおける実装上デフォルト値があるものに対してのみ、フィールドの出力を抑制するため前方互換性も担保されていると考えています。

仕様に対する変更は慎重に行うとともに、実装面に対しても互換性に関する安定性をテストの拡充などで努めて行きたいと思っていますので、今後ともご協力よろしくお願いします 🙏

[saturday06]

丁寧な解説ありがとうございます。こちらこそ、今後ともご協力させてください 🙏
ただ、次回からは何か書く前によりよく検討するよう注意します 🙇