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

Update coadd_chisquared for Python 3

    XMLWordPrintable

    Details

      Attachments

        Issue Links

        blocks

        Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7292 Port pipe_tasks to Python 3

        • Done

        Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7614 Support pytest in coadd_chisquared tests

        • Done
        is blocked by

        Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7258 Port coadd_utils to Python 3 and update unit tests for pytest

        • Done
        mentioned in

        Page Loading...

        Page Loading...

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          rowen Russell Owen added a comment - - edited

          Running on Jenkins for Python 2

          Show
          rowen Russell Owen added a comment - - edited Running on Jenkins for Python 2
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Looks good. We can remove the with_statement future imports though.

          Show
          tjenness Tim Jenness added a comment - Looks good. We can remove the with_statement future imports though.
          Hide
          Permalink
          rowen Russell Owen added a comment -

          I took a cleanup pass and fixed the following:

          • added missing setupRequired("python_future") to the ups table file
          • added tests/.cache to .gitignore as per your suggestion
          • removed unneeded imports
          • standardized import order
          • fixed a few lines that were too long
          • changed a few == None to is None

          I'll merge after a successful Jenkins run

          Show
          rowen Russell Owen added a comment - I took a cleanup pass and fixed the following: added missing setupRequired("python_future") to the ups table file added tests/.cache to .gitignore as per your suggestion removed unneeded imports standardized import order fixed a few lines that were too long changed a few == None to is None I'll merge after a successful Jenkins run

            People

            Assignee:
            rowen Russell Owen
            Reporter:
            rowen Russell Owen
            Reviewers:
            Tim Jenness
            Watchers:
            Russell Owen, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.