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

Libpredict #23

Merged
merged 115 commits into from
Feb 13, 2016
Merged

Libpredict #23

merged 115 commits into from
Feb 13, 2016

Conversation

bjorgan
Copy link
Member

@bjorgan bjorgan commented Dec 28, 2015

Orbit calculations have been replaced with calls to libpredict (latest development version) (issue #4). All unused code and most global variables have been removed, in addition to removing the server functionality (issue #3). I have also tried to clean up wherever possible (mostly Predict(), SingleTrack() and MultiTrack()), and keep the functionality and layout originally in place (and make some minor improvements here and there). It has been tested against the dummy drivers of rotctl and rigctl.

Some sacrifices have been made:

  • Lunar pass predictions miss properties: RA, Dec, GHA, Vel, Range (displays 0. Should it rather be removed altogether?). I propose to add this back after Missing properties in moon/sun calculation libpredict#49 has been dealt with, create new issue after merge.
  • Solar pass predictions miss properties: RA, Dec, GHA. See above.
  • Lon/Lat convention is hard-coded to N/E, while original version lets the user decide. I propose to add it back after resolving issue Settings variables -> struct #21, create new issue after merge.

…the satellite db entry into the function arguments.
…angle only when squint flag is set, removed daynum change throughout property displaying, resetting all variables at the beginning of orbit displaying in order to remove some AOS/LOS displaying errors when switching satellite, moved transponder name displaying so that there is no delay when switching satellites.
ryeng added a commit that referenced this pull request Feb 13, 2016
@ryeng ryeng merged commit a330e0c into master Feb 13, 2016
@bjorgan bjorgan deleted the libpredict branch February 13, 2016 17:43
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

Successfully merging this pull request may close these issues.

None yet

2 participants