
<html><head></head><body>Great, thanks for trying it out, and good catch on the failing test. Please let me know if you figure out what the problem is!<br>

<br>

Patrick<br><br><div class="gmail_quote">On 30 March 2015 15:32:16 BST, Jason Sarich <jason.sarich@gmail.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<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>

</blockquote></div></body></html>