-
Notifications
You must be signed in to change notification settings - Fork 167
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
JP-3153: post-commissioning AMI3 updates #7862
JP-3153: post-commissioning AMI3 updates #7862
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7862 +/- ##
===========================================
- Coverage 75.31% 55.01% -20.30%
===========================================
Files 474 390 -84
Lines 38965 38895 -70
===========================================
- Hits 29345 21400 -7945
- Misses 9620 17495 +7875
☔ View full report in Codecov by Sentry. |
90e5324
to
cb55336
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
783f616
to
16ff67c
Compare
16ff67c
to
4cd4257
Compare
4cd4257
to
852c5c0
Compare
852c5c0
to
f7489d9
Compare
Co-authored-by: Ned Molter <[email protected]>
Co-authored-by: Ned Molter <[email protected]>
Co-authored-by: Ned Molter <[email protected]>
…it testing at some point
…string improvements
Resolves JP-3153 and the following sub-tasks and linked tickets:
JP-1713JP-1714This PR addresses all changes to the NIRISS AMI mode analysis software since its last delivery in 2020. The main change is a complete overhaul of output formats. Two new datamodels were created: AmiOIModel (spacetelescope/stdatamodels#174) and AmiLgFitModel (spacetelescope/stdatamodels#199).
Step changes:
ami_analyze:
synphot
andstsynphot
.ami_average:
ami_normalize:
Other changes include processing 3D instead of 2D images by default for AMI exposures (rateints and calints files)
The calwebb_ami3 pipeline has been updated to run only the ami_analyze and ami_normalize steps. Association generator needs to be updated
Checklist for maintainers
CHANGES.rst
within the relevant release sectionHow to run regression tests on a PR