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

Port pipe_analysis to Python 3

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_analysis
    • Labels:
      None
    • Story Points:
      6
    • Epic Link:
    • Sprint:
      DRP F17-4, DRP F17-5
    • Team:
      Data Release Production

      Description

      As per RFC-60, the LSST stack guarantees support for Python3 and Python2.7 simultaneously. While pipe_analysis is not currently part of the official stack, the port should be done in order to ease a future integration into the main stack.

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          Technically only lsst_distrib needs to support 2.7 and 3.x. If this is not going to be part of distrib or will be integrated after spring 2018, then it can just support 3.6. Obviously if you want to support 2.7 in the mean time that's up to you, it's not required.

          Show
          tjenness Tim Jenness added a comment - Technically only lsst_distrib needs to support 2.7 and 3.x. If this is not going to be part of distrib or will be integrated after spring 2018, then it can just support 3.6. Obviously if you want to support 2.7 in the mean time that's up to you, it's not required.
          Hide
          swinbank John Swinbank added a comment - - edited

          I think we can assume that Python 2.7 support is required by the product owner, if not by the LSST bureaucracy.

          Show
          swinbank John Swinbank added a comment - - edited I think we can assume that Python 2.7 support is required by the product owner, if not by the LSST bureaucracy.
          Hide
          lauren Lauren MacArthur added a comment - - edited

          Thanks for the details. The decision can be made when the ticket rises high enough in the pecking order to get scheduled.

          Show
          lauren Lauren MacArthur added a comment - - edited Thanks for the details. The decision can be made when the ticket rises high enough in the pecking order to get scheduled.
          Hide
          sophiereed Sophie Reed added a comment -

          Hello,

          Could you look at this if you have time?

          https://github.com/lsst/obs_subaru/pull/94
          https://github.com/lsst-dm/pipe_analysis/pull/4

          Thank you

          Show
          sophiereed Sophie Reed added a comment - Hello, Could you look at this if you have time? https://github.com/lsst/obs_subaru/pull/94 https://github.com/lsst-dm/pipe_analysis/pull/4 Thank you
          Hide
          tjenness Tim Jenness added a comment -

          The PR looks good to me. There are a couple of whitespace changes that are not related to the python3 fixes so normally we'd put those on a different commit. Also one line changed from 4 cha indent to 3 and I'm not sure why.

          Show
          tjenness Tim Jenness added a comment - The PR looks good to me. There are a couple of whitespace changes that are not related to the python3 fixes so normally we'd put those on a different commit. Also one line changed from 4 cha indent to 3 and I'm not sure why.
          Hide
          sophiereed Sophie Reed added a comment -

          Hello, those whitespace changes were a mistake, sorry. I've fixed them.

          Show
          sophiereed Sophie Reed added a comment - Hello, those whitespace changes were a mistake, sorry. I've fixed them.

            People

            Assignee:
            sophiereed Sophie Reed
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Tim Jenness
            Watchers:
            John Swinbank, Lauren MacArthur, Sophie Reed, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.