<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 5, 2015 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"><br>
   Matt,<br>
<br>
    Please don't put such fixes into random branches you happen to be working on (like knepley/feature-parallel-configure). It is better to go directly into their only little branch.<br></blockquote><div><br></div><div>I thought it was fine because it was a configure branch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    Your fix doesn't work. I have pushed a fix in the branch barry/error-on-64bitindices-with-ml and merged into next. Also explained why 64 bit indices with ML are not going to happen in ml.py<br></blockquote><div><br></div><div>I accepted the email on faith.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
   self.requires32bitint  = 1;  # ml uses a combination of "global" indices that can be 64 bit and local indices that are always int therefore it is<br>
                                 # essentially impossible to use ML's 64 bit integer mode with PETSc's --with-64-bit-indices<br>
<br>
   Note: also this is the "Trilinos style" of having these two kinds of indices. I debated doing this with PETSc but felt it was too complex with little payoff.<br></blockquote><div><br></div><div>Do they mix the two kinds of indices in the interface?</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">
  Barry<br>
<br>
<br>
<br>
> On Mar 5, 2015, at 11:02 AM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
><br>
> On Thu, Mar 5, 2015 at 11:01 AM, Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>> wrote:<br>
><br>
>   Matt,<br>
><br>
>    Please ALWAYS list the branch(s)/commit that you did this in when you say you "pushed a fix"<br>
><br>
> <a href="https://bitbucket.org/petsc/petsc/commits/ba4f728f6c00f645cbd73e74de34ed390600afde" target="_blank">https://bitbucket.org/petsc/petsc/commits/ba4f728f6c00f645cbd73e74de34ed390600afde</a><br>
><br>
>   Matt<br>
><br>
>   Barry<br>
><br>
> > On Mar 5, 2015, at 9:12 AM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
> ><br>
> > I pushed a fix.<br>
> ><br>
> >    Matt<br>
> ><br>
> > On Thu, Mar 5, 2015 at 2:18 AM, Stefano Zampini <<a href="mailto:stefano.zampini@gmail.com">stefano.zampini@gmail.com</a>> wrote:<br>
> > PETSc is broken when configured with --download-ml  --with-64-bit-indices.<br>
> > ML has support for 64 bit integers (using ML's configure flag --with-ml-64bit-integer) but one needs to specify the type of 64 bit integer (long long or long or whatever...)<br>
> ><br>
> > Better to turn off 64 bit builds of ML before adding support for it during configure.<br>
> ><br>
> ><br>
> > Stefano<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>
><br>
><br>
<span class="HOEnZb"><font color="#888888">><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>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">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></div>