sot / starcheck

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

Planning t_ccd critical warning #425

Closed jeanconn closed 1 year ago

jeanconn commented 1 year ago

Description

Change the "over the planning limit" warning to critical/red .

Interface impacts

Testing

Unit tests

Independent check of unit tests by [REVIEWER NAME]

Functional tests

Output at https://icxc.cfa.harvard.edu/aspect/test_review_outputs/starcheck-pr428/

/proj/sot/ska3/flight/bin/skare starcheck -dir /data/mpcrit1/mplogs/2023/JUL3123/oflsa -out jul3123a_flight
/proj/sot/ska3/flight/bin/skare /home/jeanconn/git/starcheck/sandbox_starcheck -dir /data/mpcrit1/mplogs/2023/JUL3123/oflsa -out jul3123a_test
diff -u jul3123a_flight.txt jul3123a_test.txt > diff.txt

The diffs also show the change due to https://github.com/sot/starcheck/pull/420 as the diffs are against flight not master starcheck output, which is acceptable in this case.