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]
