Skip to content

Question: shall we update mpif90/mpirun to mpifort/mpiexec? #378

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

Closed
4 tasks done
rouson opened this issue May 8, 2017 · 3 comments
Closed
4 tasks done

Question: shall we update mpif90/mpirun to mpifort/mpiexec? #378

rouson opened this issue May 8, 2017 · 3 comments
Assignees

Comments

@rouson
Copy link
Member

rouson commented May 8, 2017

Avg response time
![Issue Stats][issue stat img]

Please replace [ ] with [X] to indicate you have taken the requested action

  • I have surrounded any code, or log output in codeblock fences
    (```) if it occupies more than one line, or in single
    backticks if it is short short code

Question:

  1. The MPI 3.1 standard mentions that many implementations provide mpirun but recommends providing mpiexec.
  2. MPI 3.1 mentions mpif90 in a similar manner but is silent on mpifort. However, comments online indicate a preference for the latter to liberate the name of the command from a specific revision of the Fortran standard.

I don't know of any implementations that don't provide mpirun or mpif90, but it seems reasonable to switch to mpifort and mpiexec for standard-conformance and clarity where the former two are employed. On the master branch, I find the following locations:

Motivation: I'm introducing caf and cafrun early the new book so this is a good time to update these commands to versions that are more standard or modern.

@zbeekman
Copy link
Collaborator

zbeekman commented Jun 7, 2017

doing this with #268

@rouson
Copy link
Member Author

rouson commented Jul 18, 2017

@zbeekman given the simplicity of this issue, shall we simply implement a quick fix rather than tie it to #268, of which this is presumably a tiny component?

@zbeekman
Copy link
Collaborator

zbeekman commented Jul 18, 2017 via email

zbeekman added a commit that referenced this issue Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants