Skip to content

Conversation

@juliohm
Copy link
Contributor

@juliohm juliohm commented Nov 12, 2021

Related to #18

This PR is a first step towards solving the manual conversion between different rotation types. After reading the code more carefully I realized that we will need to refactor existing types before we can add more conversion methods. I will open specific issues to discuss this further.

Appreciate if you can review this PR before I can continue working in other improvements.

@ronisbr
Copy link
Member

ronisbr commented Nov 17, 2021

Any idea why the tests are failing against nightly?

@juliohm
Copy link
Contributor Author

juliohm commented Dec 11, 2021

Hi @ronisbr the errors on nightly seem unrelated to the changes in the PR. Can you please double check and see if everything is ok to merge?

@ronisbr
Copy link
Member

ronisbr commented Dec 14, 2021

Hi @juliohm

I will investigate because it seems that master is passing here without problems.

@ronisbr
Copy link
Member

ronisbr commented Dec 28, 2021

Hi @juliohm

Sorry for the late response. Yes, the errors does not seem to be related to this PR, but with something that has changed with Julia-nightly.

@ronisbr ronisbr merged commit 15577a2 into JuliaSpace:master Dec 28, 2021
@ronisbr
Copy link
Member

ronisbr commented Dec 28, 2021

I also added the conversion to quaternions. The problem with Euler Angles and Euler Angle and Axis is that the type by itself does not fully define the representation. Hence, I am not sure how to implement the Julia conversion API for them.

@juliohm
Copy link
Contributor Author

juliohm commented Dec 28, 2021 via email

@ronisbr
Copy link
Member

ronisbr commented Dec 28, 2021

Yes, this will solve for Euler angles. However, for Euler angle and axis I have no idea :D

@juliohm
Copy link
Contributor Author

juliohm commented Dec 28, 2021 via email

@ronisbr
Copy link
Member

ronisbr commented Dec 28, 2021

Yes, you can always convert a rotation to an Euler angle and axis. However, the axis direction is not constant. For every DCM, for example, there is a different rotation axis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants