-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reorganizing the "Collective Communication" Chapter #90
Comments
This copy is public. |
A review was done on June 13, 2018 at the MPI meeting in Austin. Updates will be made and a reading will be presented in Barcelona. |
@puribangalore --- Please look at Ticket #80; the latest update adds _X APIs for the persistent collectives; in the process, I've had to go back and enumerate both persistent and persistent _X operations where they have failed to be named throughout Chapter 5. This has got to be related to editorial changes you've been undertaking or plan to do here. In fact, we need to correlate those changes. Ticket #105 will likewise do a review of function enumeration once we add the _X APIs later this week. |
@puribangalore --- we need to update this for the meeting next Thursday ... we need to finish before November 10! We will present in San Jose... do you need help? |
This comment has been minimized.
This comment has been minimized.
Here's the text we actually want you to read for the two-week deadline review: |
@puribangalore read the chapter; there was various feedback of a general nature. For instance, issues with commas and capitalization (already addressed to first order). However, by strawpoll, we voted to reorganize the chapter so that "All Barrier, "All Broadcast," ... "All Exscan." is the order, rather than a parallel section Blocking, Nonblocking, and Persistent. That will necessarily "transpose the 3 sections of operations, and move ahead the discussions, so that the operations will all be "together." This mirror the topologies and other chapters better. This is meant to be part of this ticket... and a careful review is required to be sure we don't change any meanings. That is not supposed to happen just by reordering presentation of the APIs. @puribangalore and @tonyskjellum will do this reordering task and present in Chattanooga in March. |
We did not do the recommended steps for Chattanooga. We are revisiting this task for Chicago. |
Aiming for MPI-4.1 because it is "text tidying." We will wait till all MPI-4.0 work is done before revisiting this. |
This needs to move very quickly if it is going to hit MPI-4.1 -- who is working on it? |
This wasn't read at the December 2022 meeting. The last opportunity for MPI 4.1 is to have it ready at the March 2022 meeting (and it needs to "pass" the reading). @tonyskjellum / @puribangalore Are you (or is someone else) planning to push this forward or should we move it out of the plan for MPI 4.1? |
This won't be ready for 4.1 but can be targeted to MPI 5.0. The title does not fit the content of the ticket anymore. The main part of the ticket which has not been handled yet is the reorganizing of the collective chapter. |
@jaegerj - You're welcome to rename the ticket to anything you like. :) |
@tonyskjellum @puribangalore would it be ok with you to rename this ticket "Reorganizing the collective chapter" ? |
Fine with me.
…On Thu, Feb 23, 2023 at 8:29 AM Julien Jaeger ***@***.***> wrote:
@tonyskjellum <https://github.com/tonyskjellum> @puribangalore
<https://github.com/puribangalore> would it be ok with you to rename this
ticket "Reorganizing the collective chapter" ?
—
Reply to this email directly, view it on GitHub
<#90 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD2CIBOIZGHBN6CUZFCFEFDWY5X4XANCNFSM4EWQV3WA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Problem
Clarifications are needed now that Persistent Communication has been added.
Proposal
Persistent communication addition has raised the need for clarifying text. This definitely applies to the Collective chapter
Changes to the Text
Chapter 5 (Collective Chapter) -- updated appropriately.
Impact on Implementations
None.
Impact on Users
Clarity for users.
References
Ticket #25 [which has passed 2 votes now] is driving this requirement, but we want to settle this concern separately as agreed in Forum meetings.
Here is the corresponding pull request: https://github.com/mpi-forum/mpi-standard/pull/89
(This is a revised pull requested as of 19-Nov-18)
The text was updated successfully, but these errors were encountered: