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

Consistency: replace "task" with "rank" or "MPI process" where appropriate #300

Closed
devreal opened this issue Jul 20, 2020 · 5 comments
Closed
Assignees
Labels
mpi-4.1 For inclusion in the MPI 4.1 standard

Comments

@devreal
Copy link

devreal commented Jul 20, 2020

Problem

The standard document spuriously uses the term task instead of rank or MPI process. The term is never defined as such and quite overloaded these days.

Proposal

For the changes for MPI 4.0 please see the PR: https://github.com/mpi-forum/mpi-standard/pull/258

Impact on Implementations

None. (I am aware that some vendors may use the term task to refer to processes. I think it's a terrible choice and not the official MPI speak.)

Impact on Users

Being more consistent avoids some distraction.

@devreal devreal self-assigned this Jul 20, 2020
@wesbland
Copy link
Member

Is this replacing the job that @dholmes-epcc-ed-ac-uk started years ago?

@dholmes-epcc-ed-ac-uk
Copy link
Member

This is a small piece of the effort by the Hybrid WG (of which I was/am a member). We clocked 2000+ instances of this type of change in the first 600 pages and then ground to a halt. Piecemeal seems like a better approach.

@wesbland wesbland added mpi-4.1 For inclusion in the MPI 4.1 standard and removed mpi-4.0 labels Oct 7, 2020
@cblaas
Copy link

cblaas commented Jan 7, 2021

I would like to see the 3 substitutions of "task" by "MPI process" in PR 258 already in MPI-4.0.

Piecemeal indeed seems to be a good approach here as these are the only three occurrences of "task" in a wrong context.

@cblaas cblaas added this to Triage in MPI 4.0 Ratification via automation Jan 7, 2021
@wgropp
Copy link

wgropp commented Jan 7, 2021

A quick check shows 21 lines that contain the word "task". Historically, as noted above, some vendors used "task" where others were using "process". And the standard was a bit sloppy about saying MPI Process (the only kind of process that the standard discusses). And while multi-threaded processes were considered from the beginning (its why the global state approach in PVM was rejected), they weren't always at the top-of-mind. If we'd caught this in MPI 1.0, we would have replaced task with process everywhere.

@wesbland wesbland moved this from Triage to In Progress in MPI 4.0 Ratification Jan 8, 2021
@wesbland wesbland removed this from In Progress in MPI 4.0 Ratification Jan 13, 2021
@wesbland
Copy link
Member

wesbland commented Mar 4, 2021

Closing this issue in favor of the chapter-specific issues created on 2021-03-04.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mpi-4.1 For inclusion in the MPI 4.1 standard
Projects
None yet
Development

No branches or pull requests

5 participants