<div dir="ltr">Jeff,<div><br></div><div>I found several issues, mainly in the error handler definition (loss of * in the errorhandler function) and datatypes creation functions (addition of "non-negative" for addresses and displacements). I have attached the output of the validation tool for all the functions I noticed. Let me know if you want it in a different format.</div><div><br></div><div>George.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 14, 2020 at 11:52 AM Jeff Squyres (jsquyres) via mpi-forum <<a href="mailto:mpi-forum@lists.mpi-forum.org">mpi-forum@lists.mpi-forum.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">



<div style="overflow-wrap: break-word;">
<div style="color:rgb(0,0,0)"><font color="#000000">The Pythonization group has Pythonized the bindings in the entire document.  We've completed a first correctness pass over the work, but we need more eyes to
 compare the end result of our work to the (non-Pythonized) head of the mpi-4.x branch.</font></div>
<div style="color:rgb(0,0,0)"><font color="#000000"><br>
</font></div>
<div style="color:rgb(0,0,0)"><font color="#000000">Some of you have already done your review.  Thank you!!</font></div>
<div style="color:rgb(0,0,0)"><br>
</div>
<div style="color:rgb(0,0,0)">To be clear: this review does not include Embiggening.  It is more-or-less the "MPI-3.1" style of function bindings.<br>
<br>
<a href="https://github.com/mpiwg-large-count/large-count-issues/wiki/pythonizing-initial-review" target="_blank">See this wiki page for instructions</a>.<br>
<br>
For additional information, also <a href="https://docs.google.com/presentation/d/1QZyh6TbjDMu7mEcNqBFfe9fFOltaCtxgZ2ZODzdtDpI/edit#slide=id.g5d01a902fe_0_69" target="_blank">see the Pythonization slides presented at the Forum in December</a>.  If you have any questions
 or problems, let us know in the Embiggening WG.</div>
<div style="color:rgb(0,0,0)"><br>
</div>
<div style="color:rgb(0,0,0)">If you just want to compare the rendered PDF:</div>
<div style="color:rgb(0,0,0)"><br>
</div>
<div style="color:rgb(0,0,0)">
<ul>
<li><font color="#000000"><a href="https://aws.open-mpi.org/~jsquyres/forum-unofficial/mpi-report-mpi4xhead-d8ad059.pdf" target="_blank">Here's a PDF from the head of the mpi-4.x branch</a> (i.e., old/original LaTeX)</font></li><li><font color="#000000"><a href="https://aws.open-mpi.org/~jsquyres/forum-unofficial/mpi-report-15b96e7-pythonized-NOT-emibggened.pdf" target="_blank">Here's a PDF from the head of the Pythonization branch</a> <i><u>with no Embiggening</u></i> (i.e,.
 new/Python-rendered LaTeX) -- updated 14 Jan 2020</font></li></ul>
</div>
<div style="color:rgb(0,0,0)"><br>
<font color="#ff0000"><span style="font-size:36px">Deadline: 21 Jan, 2020</span><br>
</font>    This gives us time to finalize everything before the T-2 week deadline for the Portland Forum meeting.</div>
<div style="color:rgb(0,0,0)"><br>
<b><u>NOTE:</u></b> At the ABQ Forum meeting, I asked you to commit/PR your approvals.json file that resulted from running the tool.  This is no longer necessary -- please just mark the google spreadsheet as described in the wiki.</div>
<div><br>
-- <br>
Jeff Squyres<br>
<a href="mailto:jsquyres@cisco.com" target="_blank">jsquyres@cisco.com</a></div>
<br>
</div>

_______________________________________________<br>
mpi-forum mailing list<br>
<a href="mailto:mpi-forum@lists.mpi-forum.org" target="_blank">mpi-forum@lists.mpi-forum.org</a><br>
<a href="https://lists.mpi-forum.org/mailman/listinfo/mpi-forum" rel="noreferrer" target="_blank">https://lists.mpi-forum.org/mailman/listinfo/mpi-forum</a><br>
</blockquote></div>