Skip to content
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

Selectable Central Body (different from Earth) #103

Open
mcontim opened this issue Feb 22, 2024 · 5 comments
Open

Selectable Central Body (different from Earth) #103

mcontim opened this issue Feb 22, 2024 · 5 comments

Comments

@mcontim
Copy link

mcontim commented Feb 22, 2024

Hi @ronisbr!

Is it possible to change the central body in SatelliteToolboxPropagators?

For my understanding, at the moment it is possible to play with the gravitational parameters passed to Propagators.init, virtually modifing the central body.

Would you think working on a more straightforward way to personalize the central body could be interesting feature to have in SatelliteToolbox?

@ronisbr
Copy link
Member

ronisbr commented Feb 26, 2024

Hi @BlueTeo91 !

All the propagators depends on some constants defined using specific structure. For example, the J2 propagator uses:

struct J2PropagatorConstants{T<:Number}
    R0::T
    μm::T
    J2::T
end

const j2c_egm2008 = J2PropagatorConstants(
    6378137.0,
    (3.986004415e14 / 6378137.0^3),
    0.0010826261738522227
)

So, in theory, you just need to use the constants related to the central body you want.

@mcontim
Copy link
Author

mcontim commented Feb 26, 2024

Thanks for your answer @ronisbr!
Indeed, at the moment I'm modifying the specific propagator structures in order to consider for a different central body.

I was wondering if you would consider adding something like CentralBody type to be passed to Propagators.init(), automatically changing the specific parameters (e.g., radius and gravitational constant) for the specific central body.
Do you think it would be an interesting enhancement or an out of scope feature for this package? Thanks

@ronisbr
Copy link
Member

ronisbr commented May 25, 2024

Hi @BlueTeo91!

Sorry for the delay, I missed that notification. Yes, probably we can add this kind of keyword to automatically select the constants if we are using other central body!

@mcontim
Copy link
Author

mcontim commented May 30, 2024

Cool @ronisbr! I can try to make a PR in the next few weeks, I would say is definitely not an urgent feature to be implemented :D

@ronisbr
Copy link
Member

ronisbr commented May 30, 2024

Thanks!!

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

No branches or pull requests

2 participants