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

Revert changes to variance in deblending

    XMLWordPrintable

    Details

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

      Description

      In DM-2914 (specifically, in 73ed9bc8) a change was made to the calculation of variance.

      This is incorrect: while deblending does split up the noise part of the pixel values as it splits up the signal, we don't want to account for that in the variance plane, because we're even more uncertain about value of deblended pixels than we are unblended pixels. Please revert it.

      This is a port of HSC-1334.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Tiny trivial cherry-pick.

            Show
            swinbank John Swinbank added a comment - Tiny trivial cherry-pick.
            Hide
            swinbank John Swinbank added a comment -

            Nate Lust – trivial review for you, please!

            Show
            swinbank John Swinbank added a comment - Nate Lust – trivial review for you, please!
            Hide
            nlust Nate Lust added a comment -

            Hi, my name is Nate Lust, and I approve of this merge

            Show
            nlust Nate Lust added a comment - Hi, my name is Nate Lust, and I approve of this merge
            Hide
            nlust Nate Lust added a comment -

            After posting this I realize you, as a Scott, may be unfamiliar with American campaign commercials, and may not get the joke here....

            Show
            nlust Nate Lust added a comment - After posting this I realize you, as a Scott, may be unfamiliar with American campaign commercials, and may not get the joke here....
            Hide
            swinbank John Swinbank added a comment -

            I haven't the faintest idea what you're talking about, but I merged anyway.

            Show
            swinbank John Swinbank added a comment - I haven't the faintest idea what you're talking about, but I merged anyway.
            Hide
            swinbank John Swinbank added a comment -
            Show
            swinbank John Swinbank added a comment - Updated release notes .

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Reviewers:
              Nate Lust
              Watchers:
              John Swinbank, Nate Lust
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.