-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add validator for sun position monitor #292
Conversation
state_name = "sun_pos_mon" | ||
msids = ["aopssupm"] | ||
plot_attrs = PlotAttrs(title="Sun position monitor", ylabel="Sun position monitor") | ||
min_violation_duration = 300 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This 300 second value is somewhat empirical from testing. It seems there is variation between the actual eclipse exit (determined by the OBC based on batteries etc) and the predicted which is in commanding. With a 300 second value most of the cases were within the limit.
This looks good to me. I note that in review of the report page I see issues with plotting negatives in the Y axis on some of the other plots which I can open as a separate issue. |
Thanks. If you are good with this then please approve it. |
Still running the unit tests. |
I think this is ready-to-go. |
Description
This adds a state validator for the
sun_pos_mon
state.It also fixes a bug where telemetry values with a trailing space character (e.g.
"ACT "
) do not match the state codes.Interface impacts
Adds a new plot in the state validation page.
Testing
Unit tests
Independent check of unit tests by Jean
Functional tests
From within the git repo:
This generated the following output: https://icxc.cfa.harvard.edu/aspect/test_review_outputs/kadi-pr292/