<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Jed,<div> I had added -pc_reuse specifically for use with fieldsplit since I wanted only a part of the preconditioner to be built only once, for e.g. -fieldsplit_0_pc_reuse. This would build the preconditioner for fieldsplit 0 only once. I had not added any API for it since it was meant to be experimental feature. If this feature is useful then I can add an API for it otherwise I can take it out.</div><div><br></div><div>Shri <br><div><div>On Mar 23, 2013, at 11:25 AM, Jed Brown wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><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>
</blockquote></div><br></div></body></html>