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
Implemented the vector transformation from one frame to another #7452
base: main
Are you sure you want to change the base?
Conversation
assert isinstance(result, u.Quantity) | ||
|
||
# Check that the result has the correct shape | ||
assert result.shape == components.shape |
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.
Should we not check the output is correct?
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.
Yes, it indeed must be checked. I was also thinking of implementing a test that examines all potential frame conversions. Would that be advantageous?
Can you let me know if this is something we should consider or if the existing single test for conversion suffices? |
I have requested for this to be reviewed and we will wait for that. |
PR Description
As requested in the #7389 this PR adds a vector transformation function in the spice module that will convert the vector from one frame to another.
TODO