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

Coordinate system clarification #566

Closed
jbherdman opened this issue Apr 17, 2016 · 3 comments
Closed

Coordinate system clarification #566

jbherdman opened this issue Apr 17, 2016 · 3 comments

Comments

@jbherdman
Copy link

In the spec, under the "Coordinate System and Units" section:

glTF uses a right-handed coordinate system, that is, the cross product of x and y yields z.

The cross product of (1,0,0) and (0,1,0) always yields (0,0,1), by mathematical construction. The distinguishing feature is that glTF uses the "right hand rule" to orient the positive-Z axis relative to the XY plane.

There must be a more concise way to state this, but with a "flat" right hand, fingers extended, if you point your fingers towards the positive-X and point your palm towards the positive-Y, then your thumb points towards the positive-Z. A "left-handed" coordinate system follows essentially the same rule, except it uses the "left hand" instead of the "right hand".

The spec could probably be improved by changing the quoted sentence to:

glTF uses a right-handed coordinate system, as is used by OpenGL.

Reference: https://www.evl.uic.edu/ralph/508S98/coordinates.html

It is probably also worth explicitly noting whether positive rotation is clockwise or counterclockwise to the axis of rotation.

@pjcozzi
Copy link
Member

pjcozzi commented Apr 25, 2016

I agree and labeled this 1.0.1 for the next minor spec update.

  • Better explain that glTF uses a right-handed coordinate system
  • Explicitly mention that positive rotation is counterclockwise.

@pjcozzi
Copy link
Member

pjcozzi commented Jun 1, 2016

Updated with e978305 in 1.0.1 branch.

@pjcozzi
Copy link
Member

pjcozzi commented Jun 15, 2017

Updated in #826

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