<div dir="ltr">Hi Patrick,<div><br></div><div>I did try out the patch, thanks. I'll push it into next, but I want to tweak it a bit. I'd feel more comfortable if we caught only the specific error that petsc4py is throwing when getsubmatrix isn't implemented. And the patch breaks my ex54mf test problem, there's a logic hole somewhere.</div><div><br></div><div>Jason</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 24, 2015 at 1:25 PM, Patrick Farrell <span dir="ltr"><<a href="mailto:patrick.farrell@maths.ox.ac.uk" target="_blank">patrick.farrell@maths.ox.ac.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="im HOEnZb">On 12/03/15 19:42, Patrick Farrell wrote:<br>
</span><span class="im HOEnZb">> I know this suggestion isn't the cleanest, but could Jason's code<br>
> try mat->ops->getsubmatrix, and if that fails fall back to the<br>
> shell matrix code?<br>
<br>
</span><div class="HOEnZb"><div class="h5">Given that (a) no one objected and (b) it solves the problem, I've implemented<br>
this suggestion in pefarrell/feature-submatrixfree-fallback of<br>
<a href="https://bitbucket.org/pefarrell/petsc" target="_blank">https://bitbucket.org/pefarrell/petsc</a>.<br>
<br>
I didn't make a pull request because for some reason the pull request shows<br>
tonnes of changes I had nothing to do with (I branched off Jason's branch,<br>
made the fix and pushed, but bitbucket seems to be confused). Jason, could you<br>
please review/cherry-pick 13006bad66cf92787ea508665d0b39f5c9b8d46f ?<br>
<br>
I'd really appreciate getting all of this stuff into next as soon as possible.<br>
It's been floating around in branches for months and I fear bitrot is setting<br>
in. My code has depended on this for months and it all works fine.<br>
<br>
Thanks,<br>
<br>
Patrick<br>
</div></div></blockquote></div><br></div>