-
Notifications
You must be signed in to change notification settings - Fork 36
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
added coordinates module, with tests #37
base: master
Are you sure you want to change the base?
Conversation
I have added the following function (with its tests):
Also, improved tests for ned2body, body2ned, wind2body, body2wind and ned2ecef functions |
Added the az_elev_dist function (with tests) |
@jorgepiloto, I have updated this PR after #39. Builds are passing except TOXENV=check |
Codecov Report
@@ Coverage Diff @@
## master #37 +/- ##
==========================================
+ Coverage 72.12% 78.30% +6.18%
==========================================
Files 11 12 +1
Lines 330 424 +94
==========================================
+ Hits 238 332 +94
Misses 92 92
Continue to review full report at Codecov.
|
fixed this PR, all test ok. @jorgepiloto |
Thank you for the pull request @aqreed and sorry for this late answer. Could you please rebase this branch onto master branch so when merging the git history looks nice? |
@jorgepiloto please tell me if everything is ok |
added the following functions for coordinates transformation: