sot / starcheck

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

Update High IR Zone check to -25 +27 around perigee #433

Closed jeanconn closed 10 months ago

jeanconn commented 10 months ago

Description

Update High IR Zone to -25 +27 around perigee

Interface impacts

Testing

I tested and ran this out of a ska3-masters (though I've forgotten where things are with what is in HEAD ska3-flight and https://github.com/sot/starcheck/pull/430 in master). Because this NMM check is really a PCAD check and the starcheck check is complementary, I don't think we need a patch/branch release for this off of starcheck 14.4.0.

As Javier noted in comments the test dependence in ska3-masters turned out to be hopper with https://github.com/sot/hopper/pull/25 .

Unit tests

Independent check of unit tests by Javier

Functional tests

Functionally, I just ran this on OCT2823A at

https://icxc.cfa.harvard.edu/aspect/SFE/2023/OCT2823/oflsa/starcheck.html

and quickly confirmed the times in the high ir zone report compared to the flight output at

https://icxc.harvard.edu/mp/mplogs/2023/OCT2823/oflsa/starcheck.html