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

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-8032 Tighten testProcessCcd thresholds once background model is fixed

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-8031 Add a test to verify the background model is created correctly

          • Won't Fix

            Activity

            Ascending order - Click to sort in descending order
            6 older comments
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            PR on meas_algorithms.

            Jenkins builds:

            Show
            swinbank John Swinbank added a comment - PR on meas_algorithms . Jenkins builds: lsst_distrib lsst_py3
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            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:

                  Jenkins

                  No builds found.