Skip to content

Renames UnitQuaternion to QuatRotation - #210

Merged
rdeits merged 2 commits into
rdeits:masterfrom
ferrolho:hf/compat-Rotations
Dec 6, 2021
Merged

Renames UnitQuaternion to QuatRotation#210
rdeits merged 2 commits into
rdeits:masterfrom
ferrolho:hf/compat-Rotations

Conversation

@ferrolho

@ferrolho ferrolho commented Dec 1, 2021

Copy link
Copy Markdown
Collaborator

@ferrolho

ferrolho commented Dec 1, 2021

Copy link
Copy Markdown
Collaborator Author

Hi, Robin! Let me know if you are happy with these changes. I asked for advice regarding one of the changes here. I chose to change the code with minimal effort, but I guess the drawback is that we are again relying on the field names used in the internal structure of quaternions.

Comment thread src/atframe.jl Outdated

@rdeits rdeits left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect, thanks!

@rdeits
rdeits merged commit e8cd09a into rdeits:master Dec 6, 2021
@ferrolho

ferrolho commented Dec 6, 2021

Copy link
Copy Markdown
Collaborator Author

Thank you, Robin! Any chance you can tag a new release to cover these changes?

@ferrolho
ferrolho deleted the hf/compat-Rotations branch December 6, 2021 13:21
@ferrolho

Copy link
Copy Markdown
Collaborator Author

Thank you, Robin! Any chance you can tag a new release to cover these changes?

@rdeits

@rdeits

rdeits commented Dec 12, 2021

Copy link
Copy Markdown
Owner

done! Sorry for the delay, we just had a baby :)

@rdeits

rdeits commented Dec 12, 2021

Copy link
Copy Markdown
Owner

@ferrolho

Copy link
Copy Markdown
Collaborator Author

done! Sorry for the delay, we just had a baby :)

That's amazing news, Robin! Congratulations!! 👶 Best of luck!

And thank you for the new version release. 😛

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