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 extrinsic to spherical and its tangent #1195
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1195 +/- ##
==========================================
- Coverage 94.32% 94.31% -0.00%
==========================================
Files 90 90
Lines 9072 9102 +30
==========================================
+ Hits 8556 8584 +28
- Misses 516 518 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Good as long as the docstring is added, with a specification of what 3D point is chosen as the origin of the theta and phi.
@@ -173,6 +174,71 @@ def tangent_spherical_to_extrinsic( | |||
|
|||
return tangent_vec_extrinsic | |||
|
|||
def extrinsic_to_spherical(self, point_extrinsic): |
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.
Add a docstring?
It would be nice to add there which point is considered the origin of the spherical coordinates.
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.
Sorry forgot! Weird that DeepSource didn't detect it though
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.
Yeah....
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.
And now adding docstrings fails codecov?
Add functions to work with spherical coordinates