Details

    • Type: Story
    • Status: Done
    • Priority: Major
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ctrl_pool
    • Labels:
      None
    • Templates:
    • Story Points:
      1
    • Sprint:
      DRP F16-4
    • Team:
      Data Release Production

      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:

              Development

                Agile