GEOS-ESM / GEOS-ESM/FVdycoreCubed_GridComp
Replace USE MAPL_ShmemMod with USE MAPL in interp_restarts.F90
Nobody has claimed this yet.
- Dominant language
- Fortran
- Stars
- 3
- Forks
- 14
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 3
Description
Problem
interp_restarts.F90 uses an internal MAPL module directly:
use MAPL_ShmemMod, only: MAPL_GetNodeInfo, MAPL_InitializeShmem, MAPL_FinalizeShmem
All MAPL references should go through the public wrapper module (USE MAPL or USE MAPL_Constants), not internal submodules.
Required Change
Merge the USE MAPL_ShmemMod symbols into the existing USE MAPL statement so that MAPL_GetNodeInfo, MAPL_InitializeShmem, and MAPL_FinalizeShmem are accessed via the public MAPL interface.
Affected Lines
interp_restarts.F90line 15 (declaration), lines 193, 194, 773 (call sites)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open interp_restarts.F90 and inspect the existing USE MAPL declaration near line 15 and the MAPL call sites near lines 193, 194, and 773. Merge the MAPL_ShmemMod symbols into the public USE MAPL statement, then verify that no direct USE MAPL_ShmemMod remains in the file.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 78/100