search

jorgepiloto / lamberthub

A set of Lambert's problem solvers
GNU General Public License v3.0
47 stars 5 forks source link

Refactor battin1984 solver #46

Closed jorgepiloto closed 3 years ago

jorgepiloto commented 3 years ago

Refactors Battin's current solver by converting one line statements into separate functions. These enables user to access all the different expressions from the original paper and will be useful when working with the gallery examples.

jorgepiloto commented 3 years ago

All tests should be passing... 🤞🏽

codecov-commenter commented 3 years ago

Codecov Report

Merging #46 (30cadee) into main (2222175) will increase coverage by 0.20%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   90.13%   90.33%   +0.20%     
==========================================
  Files          19       19              
  Lines        1226     1252      +26     
  Branches      182      182              
==========================================
+ Hits         1105     1131      +26     
  Misses         70       70              
  Partials       51       51
Impacted Files Coverage Δ
src/lamberthub/p_solvers/battin.py 100.00% <100.00%> (ø)

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 2222175...30cadee. Read the comment docs.

jorgepiloto commented 3 years ago

Merging! 🚀