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

Identify and correct differing background model between py2 and py3

    Details

      Description

      The background model pixel values differ by 0.00011 ± 0.00035 on average in python2 versus python3. The cause of this discrepancy needs to be tracked down and fixed.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -
            Show
            swinbank John Swinbank added a comment - PR on meas_algorithms . Jenkins builds: lsst_distrib lsst_py3
            Hide
            swinbank John Swinbank added a comment -

            Hey Meredith Rawls, do you have time to review this? Thanks!

            Show
            swinbank John Swinbank added a comment - Hey Meredith Rawls , do you have time to review this? Thanks!
            Hide
            mrawls Meredith Rawls added a comment -

            Looks great, just one small variable naming suggestion over at GitHub. Thank you for digging down to the root of this problem and fixing it!

            Show
            mrawls Meredith Rawls added a comment - Looks great, just one small variable naming suggestion over at GitHub. Thank you for digging down to the root of this problem and fixing it!
            Hide
            swinbank John Swinbank added a comment -

            Thanks for the review. Changes made as requested (good idea!). Waiting for Jenkins before merging:

            Show
            swinbank John Swinbank added a comment - Thanks for the review. Changes made as requested (good idea!). Waiting for Jenkins before merging: lsst_distrib: https://ci.lsst.codes/job/lsst_distrib/624/ lsst_py3: https://ci.lsst.codes/job/lsst_py3/419/
            Hide
            swinbank John Swinbank added a comment -

            Merged.

            Show
            swinbank John Swinbank added a comment - Merged.

              People

              • Assignee:
                swinbank John Swinbank
                Reporter:
                mrawls Meredith Rawls
                Reviewers:
                Meredith Rawls
                Watchers:
                John Parejko, John Swinbank, Meredith Rawls, Simon Krughoff, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel