search

kujosHeist / tle2czml

MIT License
38 stars 17 forks source link

Implement API for returning CZML string instead of writing to file #6

Closed EzraBrooks closed 4 years ago

EzraBrooks commented 4 years ago

Should close #4 by way of the tles_to_czml function.

@kujosHeist, I made these code changes ~2 years ago and only now realize I forgot to disentangle the code changes from the formatting changes (auto-formatted by pylint or autopep8, dunno which I was using at the time). If you don't want to accept the PR as-is, I can try to dissect it and/or reimplement from scratch to avoid the big whitespace changes.

kujosHeist commented 4 years ago

Hey @EzraBrooks, that's merged now thanks, I had attempted to separate out the formatting changes by committing them myself before, but didn't seem to work, no matter anyhow, I will release a new python package in the next couple of days (and will fix the six version issue as well)