[petsc-dev] Correct syntax for DMDA ownership ranges in petsc4py

[Chris Eldred]

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
<lawrence.mitchell at imperial.ac.uk> wrote:
>
>> On 8 Apr 2016, at 08:29, Chris Eldred <chris.eldred at gmail.com> 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
chris.eldred at gmail.com