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

Port obs_sdss to Python 3

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      Alert Production F16 - 9, DRP F16-6
    • Team:
      Data Release Production

      Attachments

        Issue Links

          Activity

          Hide
          swinbank John Swinbank added a comment -

          Following discussion at T/CAM standup, Fred Moolekamp has volunteered to take this on. John Parejko, I don't imagine you'll object but if you do, please shout.

          Show
          swinbank John Swinbank added a comment - Following discussion at T/CAM standup, Fred Moolekamp has volunteered to take this on. John Parejko , I don't imagine you'll object but if you do, please shout.
          Hide
          Parejkoj John Parejko added a comment -

          I'm game to let Fred do it.

          Show
          Parejkoj John Parejko added a comment - I'm game to let Fred do it.
          Hide
          fred3m Fred Moolekamp added a comment -

          The obs_sdss port to python 3 has completed but is blocked by pipe_tasks. I ran the tests requiring a connection to lsst-db on my local machine in python 2 and 3 and they all passed, as well as python 2 and 3 tests in Jenkins. However, these depend on DM-7292 (pipe_tasks port), which appears to break obs_subaru in python 2 (Jenkins does not test obs_subaru in python 3 because it has yet to be ported).

          Once DM-7292 has been merged I will put this ticket in review.

          Show
          fred3m Fred Moolekamp added a comment - The obs_sdss port to python 3 has completed but is blocked by pipe_tasks. I ran the tests requiring a connection to lsst-db on my local machine in python 2 and 3 and they all passed, as well as python 2 and 3 tests in Jenkins. However, these depend on DM-7292 (pipe_tasks port), which appears to break obs_subaru in python 2 (Jenkins does not test obs_subaru in python 3 because it has yet to be ported). Once DM-7292 has been merged I will put this ticket in review.
          Hide
          fred3m Fred Moolekamp added a comment -

          I re-ran Jenkins to confirm that it works with the most recent stack:

          1. python 2: https://ci.lsst.codes/job/stack-os-matrix/17911/
          2. python 3: https://ci.lsst.codes/job/stack-os-matrix/17910/
          Show
          fred3m Fred Moolekamp added a comment - I re-ran Jenkins to confirm that it works with the most recent stack: python 2: https://ci.lsst.codes/job/stack-os-matrix/17911/ python 3: https://ci.lsst.codes/job/stack-os-matrix/17910/
          Hide
          Parejkoj John Parejko added a comment -

          Looks good: just a few cleanups.

          Show
          Parejkoj John Parejko added a comment - Looks good: just a few cleanups.

            People

            Assignee:
            fred3m Fred Moolekamp
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            John Parejko
            Watchers:
            Fred Moolekamp, John Parejko, John Swinbank, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: