Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-28330

Update search pattern for amplifier images

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • ts_aos

    Description

      Due to https://jira.lsstcorp.org/browse/DM-28556  the raw phosim filename pattern for amplifier images has changed to   `MC_H_20000217_000032_R22_S21.fits`  and for electron images to `MC_H_20000217_006001_R22_S21_e.fits`. The `_ingestImg` function of `WEPCalculation.py` https://github.com/lsst-ts/ts_wep/blob/2f7835d4d5f3c57bd381ed1546cdcb1f5fa302b2/python/lsst/ts/wep/ctrlIntf/WEPCalculation.py#L462  needs a different search pattern to find the amplifier and e-images that need to be ingested. 

      Attachments

        Issue Links

          Activity

            moved ticket from DM-24627 to DM-28335DM-24637 is for TSSW work only... not sure if DM-28335 is the correct place.  ksuberlak - can you please place this task in the appropriate epic?  Thanks.

            aclements Andy Clements added a comment - moved ticket from DM-24627 to DM-28335 .  DM-24637 is for TSSW work only... not sure if DM-28335 is the correct place.  ksuberlak  - can you please place this task in the appropriate epic?  Thanks.

            Thank you, DM-28335 is the correct epic! 

            ksuberlak Krzysztof Suberlak added a comment - Thank you, DM-28335 is the correct epic! 

            This ticket will also include all changes to ts_wep necessitated by the update to phosim_syseng4 segmentation.txt / focalplanelayout.txt for data/lsst from DM-28557 , and by update to phosim_utils (naming scheme for the repackaged phosim images) from DM-28556

            ksuberlak Krzysztof Suberlak added a comment - This ticket will also include all changes to ts_wep necessitated by the update to phosim_syseng4 segmentation.txt / focalplanelayout.txt for data/lsst from DM-28557 , and by update to phosim_utils (naming scheme for the repackaged phosim images) from DM-28556

            All tests got updated, working on rebase for the PR .

            ksuberlak Krzysztof Suberlak added a comment - All tests got updated, working on rebase for the PR .

            This requires the obs_lsst DM-28553 to be merged, but tests also pass on the master obs_lsst (w_2021_11), since the gain values for R22 (used in tests) are similar.

            ksuberlak Krzysztof Suberlak added a comment - This requires the obs_lsst DM-28553 to be merged, but tests also pass on the master obs_lsst (w_2021_11), since the gain values for R22 (used in tests) are similar.

            Merged PR to `develop`, adding a tag as v.1.5.4

            ksuberlak Krzysztof Suberlak added a comment - Merged PR to `develop`, adding a tag as v.1.5.4
            ttsai Te-Wei Tsai added a comment -

            Chris did a great job for this adaption in ts_wep. Thanks for all the update!

            ttsai Te-Wei Tsai added a comment - Chris did a great job for this adaption in ts_wep. Thanks for all the update!

            People

              ksuberlak Krzysztof Suberlak
              ksuberlak Krzysztof Suberlak
              Te-Wei Tsai
              Andy Clements, Krzysztof Suberlak, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.