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

Use libpredict for orbit calculations #4

Closed
ryeng opened this issue Sep 18, 2015 · 2 comments
Closed

Use libpredict for orbit calculations #4

ryeng opened this issue Sep 18, 2015 · 2 comments
Assignees

Comments

@ryeng
Copy link
Contributor

ryeng commented Sep 18, 2015

The orbit calculation algorithms have been moved into libpredict (https://github.com/la1k/libpredict). Flyby should use the library instead of its own algorithms.

Until libpredict adds support for a TLE database, predict should use its own. Even after libpredict adds support, it might be useful to have a separate flyby TLE file.

@bjorgan
Copy link
Member

bjorgan commented Oct 8, 2015

I've split this issue into issue #5, #6, #7, #8, #9, #10.

Each function should be rewritten in a way that it uses no global variables related to orbit parameters or the satellite database.

Close this issue when the above issues have been resolved, and all global variables and functions related to orbit calculations can safely be removed from flyby.c.

@bjorgan
Copy link
Member

bjorgan commented Feb 13, 2016

Fixed in a330e0c

@bjorgan bjorgan closed this as completed Feb 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants