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

Update developer guide to include Python 3

    XMLWordPrintable

    Details

      Description

      Update the developer guide to indicate that Python 3 must be supported and that code must run on Python 2.7 and 3.

      This ticket will reference the tech note delivered as part of DM-6315. Writing extensive user documentation on the future package is beyond the scope of this ticket.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            My proposal is going to be along the lines that:

            • All LSST DM Python code must work with Python 3. This includes Jupyter notebooks.
            • Code with an existing user community outside of LSST must also support Python 2.7.
            • Server code and code for internal use only does not have to support Python 2.
            Show
            tjenness Tim Jenness added a comment - My proposal is going to be along the lines that: All LSST DM Python code must work with Python 3. This includes Jupyter notebooks. Code with an existing user community outside of LSST must also support Python 2.7. Server code and code for internal use only does not have to support Python 2.
            Hide
            jsick Jonathan Sick added a comment -

            That makes sense and I see no downsides. It's compatible with what SQuaRE does, too.

            This will go in the Python Style Guide, right? So it's blocked by DM-5456 at the moment?

            Show
            jsick Jonathan Sick added a comment - That makes sense and I see no downsides. It's compatible with what SQuaRE does, too. This will go in the Python Style Guide, right? So it's blocked by DM-5456 at the moment?
            Hide
            tjenness Tim Jenness added a comment -

            Yes, I didn't want to confuse things by trying to jump ahead of that ticket. I'd probably also have to tweak the modern idioms section (maybe even just removing them given that everything breaks on python 3 if you don't use them).

            Show
            tjenness Tim Jenness added a comment - Yes, I didn't want to confuse things by trying to jump ahead of that ticket. I'd probably also have to tweak the modern idioms section (maybe even just removing them given that everything breaks on python 3 if you don't use them).
            Hide
            tjenness Tim Jenness added a comment -

            Kian-Tat Lim please see the pull request for my suggested changes to the Python coding standard to support Python 3.

            Show
            tjenness Tim Jenness added a comment - Kian-Tat Lim please see the pull request for my suggested changes to the Python coding standard to support Python 3.
            Hide
            ktl Kian-Tat Lim added a comment -

            Minor comments in the PR.

            Show
            ktl Kian-Tat Lim added a comment - Minor comments in the PR.
            Hide
            tjenness Tim Jenness added a comment -

            Merged and released.

            Show
            tjenness Tim Jenness added a comment - Merged and released.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Kian-Tat Lim
              Watchers:
              Jonathan Sick, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.