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

Updated data, Add rough average # of sats, add support for N threads and N degrees with wrapper script #22

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

N6UDP
Copy link

@N6UDP N6UDP commented May 8, 2021

This includes @c2h5oh's changes in #21 with some additional changes. Unfortunately due to the line ending changes in c2h5oh's changes I based this off of Github's diff/compare tool makes main.py's changes hard to look at - I recommend using an offline diff tool to avoid looking at the whitespace only changes.

https://n6udp.github.io/starlink-coverage/ has the live version.

Additional changes:

  • Updated data as of 5/7/2021 with note (Indicate age of data #20 - this should probably be automatic), added 30 degree data
  • Added rough satellites overhead average
  • gen_cov.sh, main.py, merge_cover.py updated to support an arbitrary number of threads (Ops and process improvements #9 partial)
  • gen_cov.sh updated to generate multiple degrees in one run and to rename and move files as needed (addresses Ops and process improvements #9 to an extent)
  • Dependencies at least partially documented
  • commented out broken exclude list of sats adding some other exclusion criteria (related slightly to Formula issue #18 as are @c2h5oh's changes to whitespace)
  • gitignore

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.

2 participants