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

Smarter trajectory handling #142

Closed
cadojo opened this issue Jul 14, 2021 · 1 comment
Closed

Smarter trajectory handling #142

cadojo opened this issue Jul 14, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@cadojo
Copy link
Collaborator

cadojo commented Jul 14, 2021

Trajectories could be better!

Right now, they're just Vectors of Orbit structures. This could be so much better for many reasons, including lots of memory allocation necessary to get a high degree of precision for trajectory calculations, and more.

Ideas for Improvement

  • Allow for returning ODESolution structures from propagate with a odesolution=Val{true} switch, and manually add methods for applying astrodynamics calculations

I don't like that solution. Is there another?

OrbitalTrajectories.jl

While watching Julia-related talks online, I found the OrbitalTrajectories.jl talk. Turns out OrbitalTrajectories also uses DifferentialEquations for propagating orbits (no surprises there). What is interesting is that the ODESolution object is returned directly from the propagation. That allows for interpolating by epoch, and access to DifferentialEquations helper methods.

Conclusion

  • The Trajectory design by OrbitalTrajectories is far more robust. This should be the default, and a simple Vector of Orbit structures should be available via vec=Val{true}
@cadojo cadojo self-assigned this Jul 14, 2021
@cadojo cadojo added the enhancement New feature or request label Jul 14, 2021
@cadojo
Copy link
Collaborator Author

cadojo commented Sep 4, 2021

This was introduced in v0.10!

@cadojo cadojo closed this as completed Sep 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant