<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
Hi all,
<div class=""><br class="">
</div>
<div class="">I have done a pass through Chapter 3 looking at the Pythonization changes (a little late).</div>
<div class=""><br class="">
</div>
<div class="">I have found only one issue with the changes being made/reviewed.</div>
<div class=""><br class="">
</div>
<div class="">0) In MPI_WAITANY and MPI_TESTANY the index parameter is described as "(integer)" in both versions but should be "(non-negative integer)" in the new version. This makes it consistent with tag, source, etc. It is non-negative or MPI_UNDEFINED but
 never negative.</div>
<div class=""><br class="">
</div>
<div class="">I reported this to the BigCount WG already but please indicate (asap) if you disagree with this alteration.</div>
<div class=""><br class="">
</div>
<div class="">
<div class="">I have marked the Google spreadsheet with an ‘X’ to indicate that our review of this chapter for Pythonization is complete. Let me know ASAP if you have Pythonization-related issues.</div>
<div class=""><br class="">
</div>
</div>
<div class="">—</div>
<div class=""><br class="">
</div>
<div class="">I have also found a few things that we should deal with as separate errata.</div>
<div class=""><br class="">
</div>
<div class="">1) The LIS binding for MPI_TESTANY includes a mention of MPI_UNDEFINED in its description of the index parameter but MPI_WAITANY does not. We could fix that inconsistency either by removing the mention from the MPI_TESTANY binding or by adding
 it to the MPI_WAITANY binding. The normative text describes the use of MPI_UNDEFINED in both cases, so we would only be changing one of the LIS bindings.</div>
<div class=""><br class="">
</div>
<div class="">2) The MPI_WAITALL and MPI_TESTALL bindings for F08 give the exact size for the array_of_requests but not for the array_of_statuses, even though they must be identically sized. We could fix this by changing "(*)" to "(count)” in the rendered document
 (by including the size of the array in the Python).</div>
<div class=""><br class="">
</div>
<div class="">
<div class="">3) The F08 bindings for all variants of the MPI_TEST, MPI_WAIT, and MPI_PROBE functions and the MPI_REQUEST_GET_STATUS function do not specify an INTENT for the status(es) parameter. We could fix this by including “INTENT(OUT)” in all cases (by
 choosing the correct Python setting).</div>
<div class=""><br class="">
</div>
</div>
<div class="">4) The LIS binding for MPI_CANCEL lists the request parameter as IN but the function “marks the request for cancellation”, i.e. could modify it. This intent is implied by the C binding (the absence of const). Compare with MPI_START as well. We
 could fix this by changing “IN” to “INOUT” in the LIS binding and changing the F08 binding from “INTENT(IN)” to “INTENT(INOUT)”.</div>
<div class=""><br class="">
</div>
<div class="">I’d like to discuss these errata over email this week - if we can decide to proceed on any of them soon enough, we can create errata tickets in time for the next face-to-face, i.e. in time to be included in the MPI-4.0 document.</div>
<div class="">
<div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<br class="Apple-interchange-newline">
Cheers,</div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
Dan.</div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
—<br class="">
Dr Daniel Holmes PhD<br class="">
Architect (HPC Research)<br class="">
<a href="mailto:d.holmes@epcc.ed.ac.uk" class="">d.holmes@epcc.ed.ac.uk</a><br class="">
Phone: +44 (0) 131 651 3465<br class="">
Mobile: +44 (0) 7940 524 088<br class="">
Address: Room 2.09, Bayes Centre, 47 Potterrow, Central Area, Edinburgh, EH8 9BT</div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
—</div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
The University of Edinburgh is a charitable body, registered in Scotland, with registration number SC005336.</div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
—</div>
</div>
</div>
</div>
</div>
<br class="">
</div>
</body>
</html>