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

Port HSC-1355: Improved fringe subtraction

    XMLWordPrintable

    Details

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

      Description

      HSC-1355: "with this fix, we get much better fringe subtraction".

        Attachments

          Issue Links

            Activity

            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Sorry, please hold on about the review. I didn't run Jenkins before asking for review, and Jenkins failed. I'll investigate.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Sorry, please hold on about the review. I didn't run Jenkins before asking for review, and Jenkins failed. I'll investigate.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -
            Show
            hchiang2 Hsin-Fang Chiang added a comment - Fixed the previous problem. Jenkins passed: https://ci.lsst.codes/job/stack-os-matrix/label=centos-7/5943//console
            Hide
            swinbank John Swinbank added a comment -

            Looks good; thanks. My only comment is really tangential to the work you've done here: rather than simply labelling something a "placeholder implementation" in the code, if we know it needs to be replaced we should have a ticket for it. You didn't add the comment, but since you're touching that piece of code maybe you could take the opportunity to make sure such a ticket exists and is referred to in the comments.

            Other than that, good to merge.

            Show
            swinbank John Swinbank added a comment - Looks good; thanks. My only comment is really tangential to the work you've done here: rather than simply labelling something a "placeholder implementation" in the code, if we know it needs to be replaced we should have a ticket for it. You didn't add the comment, but since you're touching that piece of code maybe you could take the opportunity to make sure such a ticket exists and is referred to in the comments. Other than that, good to merge.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Thank you for your review! I don't think there is a ticket for expanding the fringe correction yet; I'll search and create one if none is found, and make notes in the code.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Thank you for your review! I don't think there is a ticket for expanding the fringe correction yet; I'll search and create one if none is found, and make notes in the code.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Created DM-4441 for multiple frame fringe subtraction, and added the ticket number to the comment in the code.

            Merged to master.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Created DM-4441 for multiple frame fringe subtraction, and added the ticket number to the comment in the code. Merged to master.

              People

              Assignee:
              hchiang2 Hsin-Fang Chiang
              Reporter:
              swinbank John Swinbank
              Reviewers:
              John Swinbank
              Watchers:
              Hsin-Fang Chiang, John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.