<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Nov 22, 2013 at 1:33 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>> writes:<br>
<br>
>    No, I don’t think so. XXXView() has no business calling a<br>
>    YYSetFromOptions() or YYYViewFromOptions(). In fact I go so far as<br>
>    saying MatViewFromOptions() should be removed.<br>
><br>
>  I thought we were adopting the model that only XXSetFromOptions()<br>
>  queried the database, not any old random function?<br>
<br>
</div>What if PCSetUp_LU created an empty Mat, set its prefix (to<br>
"-pc_factor_"), and called MatSetFromOptions?  Then instead of creating<br></blockquote><div><br></div><div>I agree with Barry that this should not happen. I would not object to PCSetFromOptions_Factor</div><div>
calling it on the matrix.</div><div><br></div><div>I think this rule that we only consult options from SetFromOptions should be explored. Its</div><div>certainly more modular, and makes it easy to revamp how options are consulted, etc. I am</div>
<div>guessing we will need to change our interface sometime to allow provenance information.</div><div><br></div><div>Is there a conceptual problem with storing the view options? My problem is that it seems possible</div>
<div>that you know almost nothing about how to setup at the time of SetFromOptions() and you end up</div><div>storing almost all the options.</div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

a new Mat, MatGetFactor would fill up the one that was passed in.  This<br>
seems compatible with the direction we are going in many other interfaces.<br>
<div class="im"><br>
>    I was wrong to suggest -pc_factor_mat_view, we don’t need that or want it I think.<br>
<br>
</div>I used to be able to do this from the command line.<br>
<br>
  <a href="http://scicomp.stackexchange.com/a/880/119" target="_blank">http://scicomp.stackexchange.com/a/880/119</a><br>
<br>
I don't care exactly what the options look like, but I'd like to be able<br>
to do this again.<br>
<br>
The natural place to display the factors is when they are factored<br>
rather than at the end of KSPSolve().  Note that the same factors might<br>
be used for many KSPSolves.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>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>