sot / starcheck

BSD 3-Clause "New" or "Revised" License
3 stars 0 forks source link

Trim Perl dependencies by moving more to Python #348

Closed jeanconn closed 4 years ago

jeanconn commented 4 years ago

Description

Trim Perl dependencies by moving more to Python.

Specifically, this removes the rather heavy PDL dependency and also cuts the various time-related dependencies. It also seemed time to move a bunch of Python into starcheck.utils to make it easier to call from multiple pieces of the Perl.

It also just removes the Dark Cal Checker.

Testing

Passes unit tests on MacOS, linux, Windows (at least one required) No unit tests

jeanconn commented 4 years ago

My thought regarding the Dark Cal checker is that we can either run an old version of starcheck in a pinch or resurrect the Perl module if we go back to doing those activities.

taldcroft commented 4 years ago

My thought regarding the Dark Cal checker is that we can either run an old version of starcheck in a pinch or resurrect the Perl module if we go back to doing those activities.

I'm good with cutting the dark cal checker. It is extremely unlikely we will go back to doing cals that way.

jeanconn commented 4 years ago

Great! And yeah, I figured it was fine to cut but figured I'd talk it out in PR just like we have.

jeanconn commented 4 years ago

Terrific. What would you like to see for functional testing? And indeed I should have documented. Right now I've run on recent test loads and it runs in ska3 and makes output. Since making the PR, I've also run the old complete regression test set and they run without issue (functional component) and the output looks good with the only diffs being that all the dark cal checker output is correctly missing (regression tests pass).

taldcroft commented 4 years ago

Sounds good.