This repository was archived by the owner on Jul 1, 2024. It is now read-only.
Update 2014-03-26-w3c-device-orientation-usage.md #647
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed the quaternion implementation and added some more notes about what quaternions actually are.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
I'm the author of Quaternion.js ( https://www.npmjs.com/package/quaternion ), I saw your conversion from Euler angles to quaternions uses the wrong order. The author commented he's using ZXY, but it is actually ZXY in multiplicative order and thus YXZ! and thus always rotates in the wrong order. Also the typo in the formulas made it impossible to use.
Besides that I added some more notes for people who never had contact with quaternions. Also important is that x,y,z is NOT the axis, but it only encodes the axis besides the sine! The same for the w component, it is NOT the angle, but it encodes the angle within the co-sine.
I could not fix the image with the formula, hope you can provide this.
Thanks Robert