Fix bug getting states starting between AOUPTARQ and AOMANUVR

sot / kadi

Chandra commands and events

https://sot.github.io/kadi

BSD 3-Clause "New" or "Revised" License

5 stars 3 forks source link

Fix bug getting states starting between AOUPTARQ and AOMANUVR #199

Closed taldcroft closed 3 years ago

taldcroft commented 3 years ago

Description

When the start time was between AOUPTARQ and AOMANUVR commands, this led to an exception because the code was trying to insert pseudo-commanding for that maneuver, but did not yet have a defined target attitude. This led to a failure in the maneuver generating code since it was being passed None for an attitude value.

Testing

[x] Passes unit tests on MacOS
[x] Functional testing (New unit test)

Fixes #198

taldcroft commented 3 years ago

This is a critical bug so we should get it into 2021.2.