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

HLA-1310/1311: Changes to accommodate an IR time-dependent IMPHTTAB #621

Merged
merged 3 commits into from
Sep 3, 2024

Conversation

mdlpstsci
Copy link
Contributor

@mdlpstsci mdlpstsci commented Aug 27, 2024

Allow MJD as a parameterized variable for the PHOTMODE keyword to enable a time-dependent photometric correction for the IR detector. This change supplements the change done for the UVIS in 2020 (Version 3.5.2).

Regression tests for the WFC3 IR change were successfully run here as the RAW an TRUTH files were updated appropriately: https://plwishmaster.stsci.edu:8081/job/RT/job/HSTCAL-Developers-Pull-Requests/39/consoleText

Any ACS failures are because the RAW and TRUTH files were also updates to accommodate the ACS serial CTE update where the team needs to give the final approval.

enable a time-dependent photometric correction for the IR detector.
This supplements the change done for the UVIS in 2020 (Version 3.5.2).
@mdlpstsci
Copy link
Contributor Author

IR regression tests running on Jenkins (#36)
https://plwishmaster.stsci.edu:8081/job/RT/job/HSTCAL-Developers-Pull-Requests/

@@ -39,6 +39,10 @@
for IR exposures because they're now in units of electrons.
M.De La Pena, 2020 Feb 28:
Added MJD in order to enable a time-dependent photometric correction.
M.De La Pena, 2024 Apr 15:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you mean April 15?

Copy link
Contributor Author

@mdlpstsci mdlpstsci Aug 28, 2024

Choose a reason for hiding this comment

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

@stscirij Yes. It is when I actually made the change!

Copy link
Collaborator

@stscirij stscirij 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. Why do you have to make the same history update in 3 different places (Dates, History and Updates) as well as in the code? Seems redundant...

@mdlpstsci
Copy link
Contributor Author

@stscirij
Indeed, the files are redundant at this time. This was not always true where Updates and History offer more atomic information than Dates.

@mdlpstsci mdlpstsci merged commit f240773 into spacetelescope:main Sep 3, 2024
4 checks passed
@mdlpstsci mdlpstsci changed the title HLA-1310: Changes to accommodate an IR time-dependent IMPHTTAB HLA-1310/1311: Changes to accommodate an IR time-dependent IMPHTTAB Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants