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

Adapt the Latest File Format of Corner WFS

    XMLWordPrintable

    Details

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

      Description

      Adapt the latest file format of corner WFS. This task should update the phosim_utils to repackage the amplifier images by PhoSim to mimic the output by camera control system (CCS). The example files can be found in NCSA. The path is /project/shared/BOT. The raw data is under the _parent directory. For example:

      Notice the above files are the symbolic links (created by the ingestion of obs_lsst). The file names by CCS are:

      The image name consists of 4 parts (e.g. MC_C_20200906_000371), where MC=Main Camera, C=CCS (who is in control, CCS or OCS), 20200906 = obsdate, 000371 = sequence number within obsdate. The exposure time is not in the file name, but should be in the primary header(s).

      For the difference between SW0 and SW1, need to follow the latest document: LCA-13381.

      In this task, we need to make sure the repackaged images can be ingested by obs_lsst and assembled to be two CCDs with the correct geometry compared with the eimage in PhoSim.

        Attachments

        1. image-2021-01-28-19-52-35-445.png
          37 kB
          Krzysztof Suberlak
        2. image-2021-01-28-19-52-48-114.png
          115 kB
          Krzysztof Suberlak

          Issue Links

            Activity

            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            The ticket branch has a working repackager: https://github.com/lsst-dm/phosim_utils/blob/tickets/DM-26836/python/lsst/phosim/utils/phosim_repackager.py

             

            A need for storage of test files necessitates adding a task to phosim_utils that will threshold the FITS images to reduce the file size, while allowing to add unit tests of  the repackaging of amplifier images (currently only e-image repackaging is included in test coverage). 

            Show
            ksuberlak Krzysztof Suberlak added a comment - The ticket branch has a working repackager: https://github.com/lsst-dm/phosim_utils/blob/tickets/DM-26836/python/lsst/phosim/utils/phosim_repackager.py   A need for storage of test files necessitates adding a task to phosim_utils that will threshold the FITS images to reduce the file size, while allowing to add unit tests of  the repackaging of amplifier images (currently only e-image repackaging is included in test coverage). 
            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            The PR is open at https://github.com/lsst-dm/phosim_utils/pull/12 ; all comments are being addressed

            Show
            ksuberlak Krzysztof Suberlak added a comment - The PR is open at https://github.com/lsst-dm/phosim_utils/pull/12 ; all comments are being addressed
            Hide
            ttsai Te-Wei Tsai added a comment -

            This PR looks good to me! Please do not forget to remove the ticket branch after the merge. Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - This PR looks good to me! Please do not forget to remove the ticket branch after the merge. Thanks!
            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            Marked as done, and deleted the ticket branch. Thank you for your help!

            Show
            ksuberlak Krzysztof Suberlak added a comment - Marked as done, and deleted the ticket branch. Thank you for your help!

              People

              Assignee:
              ksuberlak Krzysztof Suberlak
              Reporter:
              ksuberlak Krzysztof Suberlak
              Reviewers:
              Te-Wei Tsai
              Watchers:
              Krzysztof Suberlak, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.