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

Identify and correct differing background model between py2 and py3

    XMLWordPrintable

    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: