docs: geosynchronous phasing example#950
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #950 +/- ##
=======================================
Coverage 89.58% 89.58%
=======================================
Files 67 67
Lines 190995 190995
Branches 1428 1428
=======================================
+ Hits 171106 171108 +2
+ Misses 19474 19472 -2
Partials 415 415 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
jorgepiloto
force-pushed
the
docs/example-geo-phasing
branch
from
239a851 to
93c81c4
Compare
…ystk into docs/example-geo-phasing
|
The example should now be completed. Some of the graphics are different from #935 but all results are the same. |
jwinkle8
left a comment
jwinkle8
left a comment
There was a problem hiding this comment.
Looks great, thanks @jorgepiloto, @ldups, and @guessprojectHub for your contributions.
@ldups, I've left just a couple small comments about grammar, clarity, and cleanup.
| chaser_keplerian_elements.arg_of_periapsis = arg_of_periapsis | ||
| # - | ||
|
|
||
| # Configure the propagation segment to propagate the orbit of the chaser for 16 days: |
There was a problem hiding this comment.
Is this a copy-paste from the target satellite section? Should the explanation on line 284 mention periapsis?
There was a problem hiding this comment.
Almost- the initial conditions are identical except for the true anomaly. That might be a good thing to add to the explanation but I'm not sure exactly how to explain that...
jwinkle8
dismissed
their stale review
Comments have been addressed. Dismissing this review in case others want to review before merging.
5 participants
This pull-request adapts the code from #935 to match the layout of the examples in the repository. The code contains an example on how to perform phasing over a satellite with a geosynchronous orbit.