-
Notifications
You must be signed in to change notification settings - Fork 623
Description
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
The MPI Forum advises users that "Instead of using mpif.h, the use of the mpi_f08 or mpi module is strongly encouraged" for several reasons (https://www.mpi-forum.org/docs/mpi-3.1/mpi31-report/node411.htm) and that modernizing to use mpi[_f08] is relatively straightforward.
More practically, the current include mpif.h results in a number of Explicit interface or EXTERNAL declaration is required
compiler warnings from WW3 in compiling ufs-weather-model, like:
.../ufs-weather-model/WW3/model/src/w3parall.F90(999): warning #8889: Explicit interface or EXTERNAL declaration is required. [MPI_SEND]
CALL MPI_SEND(Iarr,1,MPI_INT, IPROC-1, 37, MPI_COMM_WAVE, IERR_MPI)
---------------^
.../ufs-weather-model/WW3/model/src/w3parall.F90(1002): warning #8889: Explicit interface or EXTERNAL declaration is required. [MPI_SEND]
CALL MPI_SEND(ipgl_tot,np_global,MPI_INT, IPROC-1, 43, MPI_COMM_WAVE, IERR_MPI)
Describe the solution you'd like
A clear and concise description of what you want to happen.
Update usage of MPI to use_mpif08 instead of include mpif.h. A branch of dev/ufs-weather-model to take an intermediate step of use_mpi is available at https://github.com/NickSzapiro-NOAA/WW3/tree/compileFix_use_mpi and is bit-for-bit in ufs-weather-model regression tests.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Disabling the warnings (questionable)
Additional context
Add any other context or screenshots about the feature request here.
Needed for ufs-community/ufs-weather-model#2703 and operational implementation
Metadata
Metadata
Assignees
Labels
Type
Projects
Status