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

Adapt ctrl_pool to py3

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ctrl_pool
    • Labels:
      None

      Attachments

        Issue Links

          Activity

          Hide
          tjenness Tim Jenness added a comment - - edited

          Fred Moolekamp changed the blocking relationship to a relates relationship between this ticket and DM-5748. I realise this is an overly pedantic discussion...

          Show
          tjenness Tim Jenness added a comment - - edited Fred Moolekamp changed the blocking relationship to a relates relationship between this ticket and DM-5748 . I realise this is an overly pedantic discussion...
          Hide
          fred3m Fred Moolekamp added a comment -

          My mistake, I changed the wrong ticket today.

          Show
          fred3m Fred Moolekamp added a comment - My mistake, I changed the wrong ticket today.
          Hide
          fred3m Fred Moolekamp added a comment -

          Using openmpi on OS X the ctrl_pool demo is able to run using n=1,3,5 in both python 2 and 3. There are currently no tests for ctrl_pool, so Jenkins can only verify that it builds correctly (see DM-3845).

          Show
          fred3m Fred Moolekamp added a comment - Using openmpi on OS X the ctrl_pool demo is able to run using n=1,3,5 in both python 2 and 3. There are currently no tests for ctrl_pool, so Jenkins can only verify that it builds correctly (see DM-3845 ).
          Hide
          tjenness Tim Jenness added a comment -

          Looks ok. Some comments on the PR. I think this may be the first python3 port involving super.

          Show
          tjenness Tim Jenness added a comment - Looks ok. Some comments on the PR. I think this may be the first python3 port involving super .
          Hide
          fred3m Fred Moolekamp added a comment -

          Rebased with master and tested the demo locally (py2 and py3) and built on Jenkins.

          Branch merged.

          Show
          fred3m Fred Moolekamp added a comment - Rebased with master and tested the demo locally (py2 and py3) and built on Jenkins. Branch merged.

            People

            • Assignee:
              fred3m Fred Moolekamp
              Reporter:
              price Paul Price
              Reviewers:
              Tim Jenness
              Watchers:
              Fred Moolekamp, John Swinbank, Paul Price, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: