Skip to content
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

Update hot ACIS observations criteria #55

Merged
merged 3 commits into from
Dec 2, 2022

Conversation

jzuhone
Copy link
Member

@jzuhone jzuhone commented Dec 1, 2022

Description

The relatively new criteria for determining "hot ACIS" observations requires the SPECTRAL_MAX_COUNT value in the obscat to be filled. If this value is not filled the obscat, the current strategy is to fall back on the "old" criteria which relied on determining the number of counts in the sequence by multiplying the expected count rate times the approved exposure.

Previously, the requirement for this older criterion was that it was only valid for observations that were S3-only or S3 with a single FI chip. SOT MP is using this criterion for all ACIS observations, regardless of which chips are used, per FSDS-53. However, acis_thermal_check is using the criterion including the chip restrictions. This PR brings acis_thermal_check in line with SOT MP's software.

Interface impacts

The number of ACIS observations that are allowed to go to -109 C as determined by this software will increase. They will show up in the table on the web pages.

Testing

Unit tests

  • No unit tests
  • Mac
  • Linux
  • Windows

Functional tests

The change to the criteria changed the answers in the unit tests. The changes to the answers were hand-checked for determine their validity.

I ran this software change on the DEC0522A load, on which the flight version of acis_thermal_check flagged a violation that should not have been flagged. The violation went away.

@jzuhone jzuhone requested review from Gregg140 and jazan12 December 1, 2022 22:25
Copy link

@jazan12 jazan12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Copy link
Contributor

@Gregg140 Gregg140 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@jzuhone jzuhone merged commit 23ca731 into acisops:master Dec 2, 2022
@jzuhone jzuhone mentioned this pull request Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants