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

Support for new brighter-fatter kernels reverted

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None

      Description

      DM-13293 created support for new-style brighter-fatter kernels, specifically in b14fc9a134e7937ea703f4b93a9a79429136d7b7.

      It was easy back then to support both the new style kernel objects, and the old, HSC-style numpy arrays, as Subaru still had its own isrTask.

      However, during the great ISR merge, this was no longer the case, and only the HSC-style kernels are now supported.

      The change was reverted here:

      https://github.com/lsst/ip_isr/commit/141792033a1ca1d5d97d72520d06508c51e9a28c#diff-3a9973baa2fe4526d09a95e405f90269L599

      We need to be able to do both.

        Attachments

          Activity

          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          The other relevant line is https://github.com/lsst/ip_isr/blob/master/python/lsst/ip/isr/isrTask.py#L882 as the new kernels are brigherFatterKernel and bfKernel refers explicitly to the HSC kernels.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - The other relevant line is https://github.com/lsst/ip_isr/blob/master/python/lsst/ip/isr/isrTask.py#L882 as the new kernels are brigherFatterKernel and bfKernel refers explicitly to the HSC kernels.
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Nice fix, thanks!

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Nice fix, thanks!
          Hide
          jchiang James Chiang added a comment -

          FWIW, I confirm that these changes work with obs_lsst.

          Show
          jchiang James Chiang added a comment - FWIW, I confirm that these changes work with obs_lsst.

            People

            Assignee:
            czw Christopher Waters
            Reporter:
            mfisherlevine Merlin Fisher-Levine
            Reviewers:
            Merlin Fisher-Levine
            Watchers:
            Christopher Waters, James Chiang, Merlin Fisher-Levine
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.