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

Enable on-sky translations for LSSTCam

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsst
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      The LSSTCam header translator does not currently support on-sky data because we don't have on-sky data and the header standardization was fluid. This is causing trouble for both Phosim and Euclid simulations. Enable RA/Dec/rotangle translations following the standard currently used by LATISS data.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          This moves some latiss methods into the shared base class so that ra/dec/rotpa can be calculated for LSSTCam.

          I have then disabled comCam those same items for comCam until we get real data to be safe.

          I have no enabled altaz calculation for LSSTCam because the test phosim file breaks if I do that. It can be trivially enabled once we get final phosim files.

          Show
          tjenness Tim Jenness added a comment - This moves some latiss methods into the shared base class so that ra/dec/rotpa can be calculated for LSSTCam. I have then disabled comCam those same items for comCam until we get real data to be safe. I have no enabled altaz calculation for LSSTCam because the test phosim file breaks if I do that. It can be trivially enabled once we get final phosim files.
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          Looks reasonable. One question about class vs. instance in two places.

          Show
          ktl Kian-Tat Lim added a comment - Looks reasonable. One question about class vs. instance in two places.

            People

            Assignee:
            tjenness Tim Jenness
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            Kian-Tat Lim
            Watchers:
            Kian-Tat Lim, Krzysztof Suberlak, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.