-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
glTF 1.1 draft #784
glTF 1.1 draft #784
Conversation
Clarify attribute semantic names in spec
Clarify attribute semantic names in spec
Clarify that parameter.node does not allow some semantics
Clarify that parameter.node does not allow some semantics
Remove unused parts of the Default Material example in Appendix A
Remove unused parts of the Default Material example in Appendix A
I don't know if there is an easier way to do this with GitHub given the history changes, but copying and pasting the 1.0 and 1.1 spec into https://www.diffchecker.com/diff has worked really well for me. |
Updated top message with TODO. I'd merge it anyway to bring everyone's attention to updated spec (we can state on the main repo page that it's a draft).
Command-line version - yes; web is a bit outdated, I'll rebuild it soon. Also we need to finish those two open spec issues. |
We can discuss today, but I would rather keep master stable and let users refer to this branch while we get a few implementations together. |
@lexaknyazev is COLLADA2GLTF updated for glTF 1.1? |
|
Added to tasklist. |
#### Orthographic projection | ||
<p><img src="figures/ortho.png" /></p> | ||
where | ||
- `r` equals `camera.orhographic.xmag`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a typo. Shouldn't this be orthographic
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. Fixed with lexaknyazev@3604243.
Is the changes to use top level arrays as opposed to dictionary already being implemented in the draft spec? |
The draft spec is not updated yet for the object to array changes. |
Looks good, except this is vague and maybe misleading:
Perhaps remove it. |
The point was that, depending on driver / API misaligned data could be rendered (because driver will fix that). |
OK, let's be more precise or remove it. |
Continues in #826. |
This PR consists of
1.0.1
branch with changed paths tospecification/1.1
directory;master
branch.Specification TODOs
Implementation TODOs
Other TODOs
resolved