Skip to content
This repository has been archived by the owner on Oct 14, 2023. It is now read-only.

A few minor improvements and WIP experiments that may be forgotten otherwise #1392

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

MLopez-Ibanez
Copy link
Contributor

A few minor improvements and WIP experiments that may be forgotten otherwise. Feel free to cherry-pick whatever you find useful. This will probably not pass the tests as a whole as it is a collection of different ideas.

@codecov
Copy link

codecov bot commented Nov 6, 2021

Codecov Report

Merging #1392 (4b8708f) into main (01a12df) will decrease coverage by 0.17%.
The diff coverage is 81.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1392      +/-   ##
==========================================
- Coverage   92.08%   91.90%   -0.18%     
==========================================
  Files          82       82              
  Lines        4346     4410      +64     
  Branches      363      365       +2     
==========================================
+ Hits         4002     4053      +51     
- Misses        255      268      +13     
  Partials       89       89              
Impacted Files Coverage Δ
src/poliastro/twobody/propagation.py 91.75% <ø> (ø)
src/poliastro/twobody/thrust/change_argp.py 100.00% <ø> (ø)
...oliastro/twobody/thrust/change_ecc_quasioptimal.py 100.00% <ø> (ø)
src/poliastro/plotting/_base.py 85.46% <50.00%> (ø)
src/poliastro/twobody/orbit.py 87.38% <57.14%> (-0.55%) ⬇️
src/poliastro/core/util.py 86.81% <78.26%> (-8.75%) ⬇️
src/poliastro/core/iod.py 90.21% <81.48%> (+0.16%) ⬆️
src/poliastro/core/elements.py 100.00% <100.00%> (ø)
src/poliastro/core/thrust/change_ecc_inc.py 100.00% <100.00%> (ø)
src/poliastro/maneuver.py 97.82% <100.00%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 01a12df...4b8708f. Read the comment docs.

@astrojuanlu
Copy link
Member

A million thanks @MLopez-Ibanez. I will have a look at this over the coming days.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants