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

Update search pattern for amplifier images

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_aos
    • Labels:

      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

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-28667 Create pipeline task framework (Im_ppg)

          • Done
          is child task of

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-28556 Adapt the Latest File Format of Corner WFS

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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.  Krzysztof Suberlak - can you please place this task in the appropriate epic?  Thanks.

            Show
            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.  Krzysztof Suberlak  - can you please place this task in the appropriate epic?  Thanks.
            Hide
            Permalink
            ksuberlak Krzysztof Suberlak added a comment -

            Thank you, DM-28335 is the correct epic! 

            Show
            ksuberlak Krzysztof Suberlak added a comment - Thank you, DM-28335 is the correct epic! 
            Hide
            Permalink
            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

            Show
            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
            Hide
            Permalink
            ksuberlak Krzysztof Suberlak added a comment -

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

            Show
            ksuberlak Krzysztof Suberlak added a comment - All tests got updated, working on rebase for the PR .
            Hide
            Permalink
            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.

            Show
            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.
            Hide
            Permalink
            ksuberlak Krzysztof Suberlak added a comment -

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

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

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

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

              People

              Assignee:
              ksuberlak Krzysztof Suberlak
              Reporter:
              ksuberlak Krzysztof Suberlak
              Reviewers:
              Te-Wei Tsai
              Watchers:
              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.