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 ACA model for chandra_models 3.34.1 #371

Merged
merged 1 commit into from
Apr 12, 2021
Merged

Update ACA model for chandra_models 3.34.1 #371

merged 1 commit into from
Apr 12, 2021

Conversation

jeanconn
Copy link
Contributor

@jeanconn jeanconn commented Apr 9, 2021

Description

This updated ACA model (md5sum 652eae461f51aaab0c78be4806bc8bce ) includes the steps for the bus 6 disable/enable/disable.

Testing

  • Functional testing

Runs fine on APR1221A and temperatures look as expected / cooler after 2021:104:12:00:00 when compared with the products used for flight review.

flight model

flight_model

new model

new_model

This includes the steps forthe bus 6 disable/enable/disable.
@jeanconn jeanconn requested a review from taldcroft April 9, 2021 15:04
Copy link
Member

@taldcroft taldcroft 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, thanks!

@jeanconn jeanconn merged commit 049060d into master Apr 12, 2021
@jeanconn jeanconn deleted the bus6 branch April 12, 2021 14:29
@javierggt javierggt mentioned this pull request Jul 2, 2021
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.

2 participants