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

Make Bright Object Masks compatible with all cameras

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Epic Link:
    • Sprint:
      DRP X16-2, DRP X16-3
    • Team:
      Data Release Production

      Description

      Currently all of the logic that goes into using bright object masks falls into obs_subaru and pipe_tasks. This ticket should move parts (such as the bright object mask class) out of obs_subaru, into a camera agnostic location. The work should also duplicate relevant camera configurations and parameter overrides in the other camera packages. Bright object masks were originally introduced in DM-4831

        Attachments

          Issue Links

          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-6031 Create documentation for bright object masks

          • To Do

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            hchiang2 Hsin-Fang Chiang added a comment -

            The code changes look fine. I was wondering if there are documentations on the LSST side about the expected format of BrightObjectMask, or how to (manually) generate BrightObjectMask?

            For a tool to generate BrightObjectMask I see https://hsc-jira.astro.princeton.edu/jira/browse/HSC-1351 on the HSC side. That sounds nice to have on the LSST side as well. (beyond this ticket, of course)

            Show
            hchiang2 Hsin-Fang Chiang added a comment - The code changes look fine. I was wondering if there are documentations on the LSST side about the expected format of BrightObjectMask, or how to (manually) generate BrightObjectMask? For a tool to generate BrightObjectMask I see https://hsc-jira.astro.princeton.edu/jira/browse/HSC-1351 on the HSC side. That sounds nice to have on the LSST side as well. (beyond this ticket, of course)
            Hide
            Permalink
            nlust Nate Lust added a comment -

            For note, the branch of obs_subaru attached to this ticket was not merged, and is only a copy of master at the current time. This was created to fix a problem in jenkins. The branch is fine to be deleted if desired.

            Show
            nlust Nate Lust added a comment - For note, the branch of obs_subaru attached to this ticket was not merged, and is only a copy of master at the current time. This was created to fix a problem in jenkins. The branch is fine to be deleted if desired.
            Hide
            Permalink
            nlust Nate Lust added a comment -

            merged to master

            Show
            nlust Nate Lust added a comment - merged to master
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            For posterity, I note that the "move parts... out of obs_subaru" was actually not necessary on this ticket: all of that work was performed on DM-4831.

            Show
            swinbank John Swinbank added a comment - For posterity, I note that the "move parts... out of obs_subaru" was actually not necessary on this ticket: all of that work was performed on DM-4831 .

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              nlust Nate Lust
              Reviewers:
              Hsin-Fang Chiang
              Watchers:
              Hsin-Fang Chiang, John Swinbank, Nate Lust
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: