<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Dec 15, 2013 at 11:35 AM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@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"><br>
On Dec 15, 2013, at 10:17 AM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
<br>
> On Sun, Dec 15, 2013 at 8:54 AM, Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>> wrote:<br>
><br>
>   Matt,<br>
><br>
>     WHEN do you want the viewing to be done? That is where the DMViewFromOptions() should be called, and you should move it there. Surely you don’t wish to view the DM in DMSetFromOptions() where presumably nothing is setup?   Note that in analogy, MatViewFromOptions() is called in MatAssemblyEnd().<br>

><br>
>     Since we are going to allow XXXViewFromOptions() to be turned off with a master switch for performance, we can just put it were we want it and don’t need to call it one place and then use the resulting viewer somewhere else.<br>

><br>
> For now, I would like to remove it from DMSetUp() where it has no meaning, and just make the user put it in manually.<br>
> Is that alright?<br>
<br>
</div>   Yes, it can also go into the end of DMSetUp_DA().<br>
<br>
    Is there some concept of a DMAssemblyEnd() stage where one knows that the DM truly is ready to go? Should there be?</blockquote><div><br></div><div>I guess we need to put that in. Since I did not allow any communication, I left it out.</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"><span class="HOEnZb"><font color="#888888"><br>
   Barry<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
>    Thanks,<br>
><br>
>       Matt<br>
><br>
><br>
>    Barry<br>
><br>
> On Dec 14, 2013, at 11:15 PM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
><br>
> > The move to DMSetUp() has broken every DMPlex test. The full mesh<br>
> > is usually not constructed at DMSetUp(), which just allocates memory.<br>
> > This is analogous to MatSetUp(), for which we do not expect values to<br>
> > be set.<br>
> ><br>
> > I don't think the ViewFromOptions call makes sense in DMSetUp() and<br>
> > think it should be moved back to SetFromOptions.<br>
> ><br>
> >    Matt<br>
> ><br>
> > --<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<br>
><br>
><br>
><br>
><br>
> --<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<br>
<br>
</div></div></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>