<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 22, 2013 at 10:14 AM, Mark F. Adams <span dir="ltr"><<a href="mailto:mark.adams@columbia.edu" target="_blank">mark.adams@columbia.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>I thought the primary mission of options_left was to catch spelling mistakes.</div>
<div class="im"><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra">(This is also useful if a PC logically ought to pay attention to this option, but isn't.) What about making PCSetUseAmat() use PetscTryMethod() and move the check of "-pc_use_amat" into each implementation?</div>

<div class="gmail_extra"><br></div></div></blockquote><div><br></div></div><div>I agree with Barry that it should be high level and make it there for any PC to use if they want.</div><div class="im"><br><blockquote type="cite">
<div dir="ltr"><div class="gmail_extra">Do we really want to leave the confusing option in place for all the PCs that don't have inner solvers?</div>
</div>
</blockquote></div></div><br><div>It just doesn't seem that bad to me.</div></blockquote></div><br>Okay, okay, I'll stick with the simple thing.</div><div class="gmail_extra"><br></div><div class="gmail_extra">What about "-pc_reuse"? It is only referenced in src/ksp/pc/impls/factor/lu/lu.c and it doesn't have an API. Can it be removed entirely? I thought we were using SAME_PRECONDITIONER for this sort of thing.</div>
</div>