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

Add CR morphology code.

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None

      Description

      Do we have a ticket to check the morphology of CRs detected on dark exposures? We're seeing problems on LATISS due to missing VBB, and that would catch it

      `ip_isr` is a reasonable place to put the kind of things that Merlin implemented in his notebook.  I'm thinking about the isrQa.py code, where this can be added as an optional calculation

        Attachments

          Issue Links

            Activity

            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            I don't disagree with this at all, but annoyingly, the current CR code (detection and interpolation) is in imChar (because of "needing" a PSF model), so there is that to consider.

            I 100% agree that for QA of sensor stuff that isr is the right/nice to place for it to live, especially as running imChar for this is long and relatively expensive, and a bit of an abuse of the task, as you have to turn everything off otherwise it will fail horribly on darks, not to mention means the metrics will then live in the wrong place...

            Anyway, no real point there, just wanted to note that on the ticket.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - I don't disagree with this at all, but annoyingly, the current CR code (detection and interpolation) is in imChar (because of "needing" a PSF model), so there is that to consider. I 100% agree that for QA of sensor stuff that isr is the right/nice to place for it to live, especially as running imChar for this is long and relatively expensive, and a bit of an abuse of the task, as you have to turn everything off otherwise it will fail horribly on darks, not to mention means the metrics will then live in the wrong place... Anyway, no real point there, just wanted to note that on the ticket.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              czw Christopher Waters
              Watchers:
              Andrés Alejandro Plazas Malagón, Christopher Waters, Merlin Fisher-Levine, Robert Lupton
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.