<div dir="ltr"><div dir="ltr">On Sat, Feb 9, 2019 at 3:17 AM Smith, Barry F. via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov">petsc-dev@mcs.anl.gov</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
  Hmm, why is this check wrong (though it may be incomplete)? Is there an example where it triggers an error when it shouldn't?<br>
<br>
   It is my understanding that the check is there to make sure if the user requested a particular matrix type then it won't return a different type that is unexpected for the user (the AIJ checks are just to handle MATAIJ vs MATSEQAIJ, MATMPIAIJ).<br></blockquote><div><br></div><div>First, the logic is hard to read. It checks if A is mattype. I would think it would error if this is not true. However,</div><div>it further checks whether its SEQAIJ or MPIAIJ, and only errors if it is not. This seems to say, if the use specifies</div><div>a mattype, like CUSPARSE, but we give back AIJ, everything is alright? I am reading this wrong?</div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
   Barry<br>
<br>
<br>
> On Feb 8, 2019, at 5:29 AM, Stefano Zampini via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov" target="_blank">petsc-dev@mcs.anl.gov</a>> wrote:<br>
> <br>
> I don’t understand the need of these checks (tacky and incomplete…, as per the comment) <br>
> <br>
> <a href="https://bitbucket.org/petsc/petsc/src/75aeaa18afd73be7f3020f94b788241b9e0ba3d7/src/dm/impls/shell/dmshell.c#lines-197" rel="noreferrer" target="_blank">https://bitbucket.org/petsc/petsc/src/75aeaa18afd73be7f3020f94b788241b9e0ba3d7/src/dm/impls/shell/dmshell.c#lines-197</a><br>
> <br>
> Why are these checks needed? I’m inclined to remove these lines, but I would like some comments before doing it.<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div><div><br></div><div><a href="http://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br></div></div></div></div></div></div></div></div>