Correct CRC and better error handling

nsat / pypredict

Spire port of predict open-source tracking library

GNU General Public License v2.0

64 stars 17 forks source link

Correct CRC and better error handling #41

Open jenshz opened 3 years ago

jenshz commented 3 years ago

While trying to use TLEs from Celestrak I encountered a parse error. I've updated the KepCheck to checksum_tle from https://github.com/philcrump/libpredict, and have added some extra error handling (so you can actually figure out what's wrong with your TLE).

nick-pascucci-spire commented 2 years ago

Hi, thanks for your pull request! This looks like it will be a helpful fix.

Before I merge your PR I'd like to make sure it's tested. Would you mind adding some test cases showing failures for each case with malformed TLEs and a success with a well-formed one?