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

Investigate JSON Schema 4 #252

Closed
pjcozzi opened this issue Mar 12, 2014 · 5 comments
Closed

Investigate JSON Schema 4 #252

pjcozzi opened this issue Mar 12, 2014 · 5 comments

Comments

@pjcozzi
Copy link
Member

pjcozzi commented Mar 12, 2014

The glTF schema is written using JSON Schema 3.

I'll look into JSON Schema 4. We'll use it if it adds value, which will mean switching the validator from JSV to JJV (or another validator that supports Schema 4).

@pjcozzi
Copy link
Member Author

pjcozzi commented Mar 13, 2014

There's a list of validators here. Several support Schema 4.

Schema 5 is also in the works.

@pjcozzi pjcozzi added this to the Draft 1.0 spec milestone Apr 30, 2014
@pjcozzi pjcozzi mentioned this issue Apr 30, 2014
8 tasks
@pjcozzi pjcozzi removed this from the Spec 1.0 milestone Aug 27, 2015
@pjcozzi
Copy link
Member Author

pjcozzi commented Sep 14, 2015

I don't plan on moving from JSON Schema 3 to 4 right away, but here's:

@sbtron
Copy link
Contributor

sbtron commented Nov 2, 2016

Wanted to hit this up again as we are looking into implementing a native glTF parser using rapidJSON. rapidJSON needs JSON Schema 4. I can work on updating to the schema needed to go from 3 to 4. Are there any other updates needed around validators etc. if the schema was updated from 3 to 4?

@pjcozzi
Copy link
Member Author

pjcozzi commented Nov 2, 2016

This would be fantastic.

For some examples using JSON Schema 4, check out how we use it for 3D Tiles (including validation):

https://github.com/AnalyticalGraphicsInc/3d-tiles/tree/master/schema#3d-tiles-json-schema

It would not be required right away, but we would also very much welcome a contribution to update wetzel:

https://github.com/AnalyticalGraphicsInc/wetzel

which is a small Node.js tool I wrote to generate the property references in the spec from the schema.

There is also a C# loader that was generated from the schema that we would have to determine the best path forward for; I'm not sure if it is being maintained:

https://github.com/KhronosGroup/glTF/tree/master/loaders/CSharp

@pjcozzi
Copy link
Member Author

pjcozzi commented Jul 16, 2017

Fixed in #769 and friends

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants