search

PerezHz / PlanetaryEphemeris.jl

Solar System ephemeris Taylor integrator based on JPL DE430/431 dynamical model
Other
20 stars 6 forks source link

Propagation output: Change units from sec -> au, au/day to day -> au, au/day #19

Closed PerezHz closed 1 year ago

PerezHz commented 1 year ago

Related to PerezHz/NEOs.jl#17. I would propose to make this change here; it can actually save us the unit conversion step after propagation here, and then users can always use auday2kmsec and related functions to change ephemeris units if needed. What do you think, @lbenet, @LuEdRaMo?

LuEdRaMo commented 1 year ago

I'm in favor of this change; as you mention, it can save us a conversion here and the loadeph function at NEOs.jl. Should we address this before https://github.com/PerezHz/PlanetaryEphemeris.jl/issues/20?.

PerezHz commented 1 year ago

This change should be easier to pursue than fixing #20, so yes I'd propose to first tackle the issue here.

PerezHz commented 1 year ago

Fixed by #22