On Tue, May 18, 2010 at 4:25 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
  Yes it could do with some love, but that love is not coming from me.</blockquote><div><br></div><div>Well, I am waiting for someone else to use it. All the examples from Spiegelman</div><div>run correctly and he uses it. Yes, I tried to make it more general than DAs, but</div>
<div>it is incredibly difficult in parallel, so I gave up. If someone thinks they know how,</div><div>we can change it.</div><div><br></div><div>We could take out dead code I guess.</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;"><font color="#888888"><br>
    Barry<br>
</font><div><div></div><div class="h5"><br>
On May 18, 2010, at 4:13 PM, Jed Brown wrote:<br>
<br>
> I noticed the shared library fixes go in the other day (along with it<br>
> moving out of contrib a while back).  As a purely organizational matter,<br>
> shouldn't this thing be at src/characteristic?  There are some dubious<br>
> unreachable, #if 0, and debugging (example below) code fragments hanging<br>
> around.<br>
><br>
>        printf("[%d]Remote point (%d) at n+1/2 to neighbor %d: (i:%d, j:%d) (x:%g, y:%g)\n", rank, (int)c->queueSize+1, Qi.proc, Qi.i, Qi.j, Qi.x, Qi.y);<br>
><br>
> There is also DA-specific code in the interface despite appearances that<br>
> the DA-specific part is isolated to impls/da.  It would appear that this<br>
> package could use some love if it's going to be widely used (which is<br>
> one possible interpretation of the significance of moving out of<br>
> contrib).<br>
><br>
> Jed<br>
<br>
</div></div></blockquote></div><br><br clear="all"><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>