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

Add Transform3DF32::from_quaternion #111

Closed

Conversation

@asajeffrey
Copy link
Member

asajeffrey commented Mar 20, 2019

Add a function to build a transform from a quaternion.

@pcwalton
Copy link
Collaborator

pcwalton commented Mar 20, 2019

Needs SIMD

@asajeffrey
Copy link
Member Author

asajeffrey commented Mar 20, 2019

Why does this need SIMD but, e.g, from_ortho doesn't? If it makes any difference, the way this is going to be used it won't be on a hot path. We could check it in with a TODO: SIMD?

@pcwalton
Copy link
Collaborator

pcwalton commented Mar 20, 2019

The code will be clearer with SIMD. Give me a few minutes and I'll make a go of it.

@pcwalton
Copy link
Collaborator

pcwalton commented Mar 20, 2019

(I'd also prefer to use rotors instead of quaternions, but I'm OK with deferring that.)

pcwalton added a commit that referenced this pull request Mar 20, 2019
pcwalton added a commit that referenced this pull request Mar 20, 2019
@pcwalton
Copy link
Collaborator

pcwalton commented Mar 20, 2019

I pushed this patch, partially SIMD-ified: pcwalton@26fea26

(It's untested but I triple checked it.)

Thanks for the patch!

@pcwalton pcwalton closed this Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.