Great! How would I go about testing that change- I have been using the petsc4py version that comes with Firedrake, so what is the correct way to modify petsc4py and rebuild it? On Fri, Apr 8, 2016 at 10:01 AM, Lawrence Mitchell <[email protected]> wrote:

...

On 8 Apr 2016, at 08:29, Chris Eldred <[email protected]> wrote:

Yes I have tried all possible variants: [(8,8,4),] ([8,8,4],) [[8,8,4],] ((8,8,4),) and they all fail with the same TypeError: Expected tuple, got list

However, I think I see what is happening now. In line 156 cdef object ranges = list(ownership_ranges) it looks like ranges is being defined as a list, rather than a tuple. So maybe there should be a cast from list to tuple in the return statement?

That looks right, since the return type of asOwnerShipRanges is declared as tuple, but as you note, the return variable has type list.

Lawrence

-- Chris Eldred Postdoctoral Fellow, LAGA, University of Paris 13 PhD, Atmospheric Science, Colorado State University, 2015 DOE Computational Science Graduate Fellow (Alumni) B.S. Applied Computational Physics, Carnegie Mellon University, 2009 [email protected]