Skip to content
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

生成シリアライザをデフォルトにする #564

Merged
merged 6 commits into from
Sep 28, 2020

Conversation

ousttrue
Copy link
Contributor

#553
#490

https://github.com/KhronosGroup/glTF-Validator/blob/1be41dd1acdfe21ee9e577577bc2183e6861345a/ISSUES.md

  • BUFFER_VIEW_TARGET_OVERRIDE
  • ACCESSOR_MAX_MISMATCH

は、validator の方に 修正が要ると思う。

https://github.com/KhronosGroup/glTF-Sample-Models

に該当するサンプルを送って修正依頼を試みる。
(違うタイプの sparse しかサンプルが無い )

  • Microsoft 3D Viewer
  • Three.js
  • babylon.js

での動作を確認した。

@saturday06
Copy link
Contributor

シリアライザーやデシリアライザーの変更は事故る確率が他の変更よりも高いと思っているため、こちらがマージされてもすぐにリリースせず一週間程度寝かせる期間を置いてもらえると嬉しいです。

@ousttrue ousttrue added this to the v0.61 milestone Sep 28, 2020
@ousttrue ousttrue merged commit fbfae2c into vrm-c:master Sep 28, 2020
@ousttrue
Copy link
Contributor Author

10/5 の週にリリースする予定です。

https://github.com/vrm-c/UniVRM/milestone/20

@saturday06
Copy link
Contributor

対応ありがとうございます!

@ousttrue
Copy link
Contributor Author

ousttrue commented Oct 1, 2020

validator にひっかかるデータを作りました。

KhronosGroup/glTF-Sample-Models#274

@ousttrue ousttrue deleted the feature/use_serializertypes_generated branch April 2, 2021 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants