sot / starcheck

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

Allow using maude for thermal model source data #397

Closed jeanconn closed 1 year ago

jeanconn commented 1 year ago

Description

Allow using MAUDE for thermal model source data, with automatic selection of MAUDE if eng_archive data is not available. This is handy on my laptop as I no longer bother with syncing the archive. Adds an option to use maude for other systems when more up-to-date telemetry is desired.

While making it work on the laptop, I also noticed that the plotting routing was always using miniagasc.h5 instead of the supplied agasc_file, so added a fix for that.

Interface impacts

Machines without AACCCDPT in their cheta archive will fallback to using MAUDE/maude for initial conditions for the ACA thermal model. Also adds the "-maude" option to allow users to use MAUDE/maude if preferred (but does not change default unless the archive is missing or broken).

Testing

Unit tests

Functional tests

Ran on my OSX laptop fine with AACCCDPT no longer in eng_archive.

On fido tested with and without maude option and compared the without option to flight with expected results:

/home/jeanconn/git/starcheck/sandbox_starcheck -dir /data/mpcrit1/mplogs/2022/DEC1922/oflsa -maude -out dec1922t_maude

https://icxc.cfa.harvard.edu/aspect/test_review_outputs/starcheck-pr397/dec1922t_maude.html

/home/jeanconn/git/starcheck/sandbox_starcheck -dir /data/mpcrit1/mplogs/2022/DEC1922/oflsa -out dec1922t_cxc

https://icxc.cfa.harvard.edu/aspect/test_review_outputs/starcheck-pr397/dec1922t_cxc.html

starcheck -dir /data/mpcrit1/mplogs/2022/DEC1922/oflsa -out dec1922_flight

https://icxc.cfa.harvard.edu/aspect/test_review_outputs/starcheck-pr397/dec1922_flight.html

diff

https://icxc.cfa.harvard.edu/aspect/test_review_outputs/starcheck-pr397/diff.html

jeanconn commented 1 year ago

I updated per comments and reran the testing. I note the testing no longer shows much difference between the telemetry sources in the temperature plot as the DEC1922 loads are well underway.