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

Make phosim 3.9 wavefront sensing chips work with obs_lsst

    Details

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

      Description

      The wavefront sensing chips do not run through runIsr.py with master of obs_lsst (housed in the obs_lsstCam repository. Suggest changes to make that happen.

        Attachments

          Activity

          krughoff Simon Krughoff created issue -
          krughoff Simon Krughoff made changes -
          Field Original Value New Value
          Epic Link DM-14794 [ 122961 ]
          krughoff Simon Krughoff made changes -
          Risk Score 0
          Hide
          krughoff Simon Krughoff added a comment -

          Merlin, this is very small and hopefully non-controversial.

          Show
          krughoff Simon Krughoff added a comment - Merlin, this is very small and hopefully non-controversial.
          krughoff Simon Krughoff made changes -
          Reviewers Merlin Fisher-Levine [ mfisherlevine ]
          Status To Do [ 10001 ] In Review [ 10004 ]
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Or so you thought!

          Sorry for being so picky, but hopefully each comment will make sense.

          Feel free to ignore the general "why is this necessary" comment at the end if you want to, it was mainly just curiosity, but I never saw that explained in the rooms where it was being discussed, and I'm just curious how/why that made it in in the first place.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Or so you thought! Sorry for being so picky, but hopefully each comment will make sense. Feel free to ignore the general "why is this necessary" comment at the end if you want to, it was mainly just curiosity, but I never saw that explained in the rooms where it was being discussed, and I'm just curious how/why that made it in in the first place.
          mfisherlevine Merlin Fisher-Levine made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Hide
          krughoff Simon Krughoff added a comment -

          Merged.

          Show
          krughoff Simon Krughoff added a comment - Merged.
          krughoff Simon Krughoff made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          tjenness Tim Jenness made changes -
          Component/s obs_lsst [ 16504 ]
          Component/s obs_lsstCam [ 15215 ]

            People

            • Assignee:
              krughoff Simon Krughoff
              Reporter:
              krughoff Simon Krughoff
              Reviewers:
              Merlin Fisher-Levine
              Watchers:
              Merlin Fisher-Levine, Simon Krughoff
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel