On Thu, Jan 5, 2012 at 4:07 AM, Dominik Szczerba <span dir="ltr">&lt;<a href="mailto:dominik@itis.ethz.ch">dominik@itis.ethz.ch</a>&gt;</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">
Is it also possible that MaxAbs would be computed wrongly? I get a<br>
different value when I compute it myself.</blockquote><div><br></div><div>Yes, that is possible. Here is the code:</div><div><br></div><div>  <a href="http://petsc.cs.iit.edu/petsc/petsc-dev/annotate/d1799f05bf01/src/mat/impls/aij/seq/aij.c#l2628">http://petsc.cs.iit.edu/petsc/petsc-dev/annotate/d1799f05bf01/src/mat/impls/aij/seq/aij.c#l2628</a></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>
Dominik<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Thu, Jan 5, 2012 at 1:02 AM, Barry Smith &lt;<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>&gt; wrote:<br>
&gt;<br>
&gt; On Jan 4, 2012, at 5:42 PM, Matthew Knepley wrote:<br>
&gt;<br>
&gt;&gt; On Wed, Jan 4, 2012 at 5:29 PM, Dominik Szczerba &lt;<a href="mailto:dominik@itis.ethz.ch">dominik@itis.ethz.ch</a>&gt; wrote:<br>
&gt;&gt; Do you think this issue can potentially explain a behavior I observe:<br>
&gt;&gt;<br>
&gt;&gt; In my big block shell matrix there are two blocks submatrices: (say) B<br>
&gt;&gt; and B transposed. MaxAbs is exact same for both submatrices, MinAbs is<br>
&gt;&gt; zero for B (as expected), but small negative for the other<br>
&gt;&gt; (unexpected).<br>
&gt;&gt;<br>
&gt;&gt; Now question: do I have a bug (that I do not immediately see) or this<br>
&gt;&gt; can be hopefully only a trick in MinAbs?<br>
&gt;&gt;<br>
&gt;&gt; It was a bug in MinAbs. I am pushing the fix.<br>
&gt;<br>
&gt;    To 3.2 I hope. Please send the patch link for 3.2 as Satish did just a little while ago for another bug.<br>
&gt;<br>
&gt;   Barry<br>
&gt;<br>
&gt;&gt;<br>
&gt;&gt;    Matt<br>
&gt;&gt;<br>
&gt;&gt; Thanks a lot.<br>
&gt;&gt;<br>
&gt;&gt; On Wed, Jan 4, 2012 at 6:05 PM, Jed Brown &lt;<a href="mailto:jedbrown@mcs.anl.gov">jedbrown@mcs.anl.gov</a>&gt; wrote:<br>
&gt;&gt; &gt; On Wed, Jan 4, 2012 at 11:02, Matthew Knepley &lt;<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>&gt; wrote:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Okay, here is what is wrong with the logic:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; 1) Its not a shift, it ignores values &lt; 1.0e-12<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; you wrote that in the commit message<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; 2) The problem is on line 2.17 of the diff where it takes the first value<br>
&gt;&gt; &gt;&gt; as minimum, but does not take the absolute value<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; good catch<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
&gt;&gt; -- Norbert Wiener<br>
&gt;<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<br>