Skip to content
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

Clarification: Section 11.7 - p. 482 - MPI Exception Clarification #263

Closed
wesbland opened this issue Feb 19, 2020 · 6 comments
Closed

Clarification: Section 11.7 - p. 482 - MPI Exception Clarification #263

wesbland opened this issue Feb 19, 2020 · 6 comments
Assignees
Labels
errata Errata items for the previous MPI Standard passed final vote Passed the final formal vote wg-ft Fault Tolerance Working Group wg-rma RMA Working Group

Comments

@wesbland
Copy link
Member

wesbland commented Feb 19, 2020

Problem

"MPI exception" is used, but this term is not well defined (and the usage is the "signal an MPI exception" - MPI doesn't signal.

Suggested Fix

Chapter committee to update

Associated pull request

https://github.com/mpi-forum/mpi-standard/pull/177

@wesbland wesbland added editor pass wg-rma RMA Working Group and removed mpi-4.x labels Feb 19, 2020
@wesbland
Copy link
Member Author

wesbland commented Apr 8, 2020

This is probably an errata as well, but needs discussion:

  • Editor - 1
  • Chapter committee - 2
  • Errata - 3
  • Full Proposal - 1

@abouteiller
Copy link
Member

MPI Exception is defined in 8.3. The more common form for the problematic text in the RMA chapter is the call raises an MPI error.

@abouteiller abouteiller reopened this Apr 10, 2020
@wesbland wesbland self-assigned this Apr 13, 2020
@wesbland wesbland added wg-ft Fault Tolerance Working Group errata Errata items for the previous MPI Standard labels Apr 13, 2020
@wesbland wesbland added this to Awaiting Informal Reading in Editor Pass Status Apr 13, 2020
@wesbland wesbland moved this from Awaiting Informal Reading to Finished Informal Reading in Editor Pass Status Apr 15, 2020
@jsquyres
Copy link
Member

jsquyres commented Jul 1, 2020

MPI-3.1:8.3 p340:15 says: "Each such error generates an MPI exception." (and "MPI exception" is actually bold faced in the spec).

Are you removing all cases of "MPI exception" from the spec except for the place that initially names it? I'm confused as to what problem this issue is trying to solve...?

@wesbland
Copy link
Member Author

wesbland commented Jul 6, 2020

Yes. That's a good point. We missed that (and a few other places) before. We made some more changes today in the WG to fix that.

The main goal is to clarify the terms failure, exception, and error, by removing the term "exception" from the MPI side of things (there are still OS exceptions, etc.). So now we don't "signal an exception", "raise an exception", or anything else. We just "raise an error".

@wesbland
Copy link
Member Author

wesbland commented Jul 11, 2020

This failed a "no no" vote on 2020-07-01.

The subsequent errata vote was withdrawn.

https://www.mpi-forum.org/meetings/2020/06/votes

@wesbland wesbland added the scheduled reading Reading is scheduled for the next meeting label Jul 11, 2020
@wesbland
Copy link
Member Author

This passed a "no no" and errata vote on 2020-08-19.

https://www.mpi-forum.org/meetings/2020/08/votes

@wesbland wesbland added passed final vote Passed the final formal vote and removed scheduled reading Reading is scheduled for the next meeting labels Aug 24, 2020
@wesbland wesbland closed this as completed Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
errata Errata items for the previous MPI Standard passed final vote Passed the final formal vote wg-ft Fault Tolerance Working Group wg-rma RMA Working Group
Projects
No open projects
Editor Pass Status
  
Finished Informal Reading
Development

No branches or pull requests

3 participants