-
Notifications
You must be signed in to change notification settings - Fork 14
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 quaternions page #39
Add quaternions page #39
Conversation
@KronosTheLate Do you have any thoughts on this PR? Should we wait for some other reviews? |
@moble Hi, this PR adds a new comparison page for quaternions. I mentioned you because you are maintaining Quaternionic.jl. Feel free to comment if my comparison is misleading or incorrect. (sorry for lacking an explanation in the first comment!) |
Yep, I actually started a review when I saw your first comment but didn't finish it. :) Will finish it in the next hour or so. |
I have to admit, I am in the middle of some quite intense exam preparations. But I do not expect development to stop only for my sake. If we can get you both as independent actors to "sign off" on this PR, I am all good with merging it. The PR author has obviously approved it, so as soon as @moble gives the green light, we have two independant thumbs up, and @hyrodium is free to merge it. |
Co-authored-by: Mike Boyle <[email protected]>
Co-authored-by: Mike Boyle <[email protected]>
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.
Looks good. Thanks!
Awesome! Just skimming the page, I already learnt a think or two about quaternions and their state in the Julia package ecosystem. Thanks <3 |
This PR fixes #38.
(cc: @moble)