pypredict icon indicating copy to clipboard operation
pypredict copied to clipboard

Published 20 hours ago • verified publisher icon nsat

Correct CRC and better error handling

Open jenshz opened this issue 3 years ago • 1 comments

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).

jenshz avatar Jan 24 '21 14:01 jenshz

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?

nick-pascucci-spire avatar Feb 03 '22 10:02 nick-pascucci-spire