Recommended Issues to Address from PR #22
Opened this issue · 2 comments
Describe the bug
I am opening this issue to document the recommended changes from Rusty's review of PR #22 after PR 22 is merged.
The following are comments copied from the PR:
SHiELD_physics/GFS_layer/GFS_typedefs.F90
Lines 2834 to 2841 in 6a6e94f
From bensonr: Stop statements may not abort a program correctly when using MPI. Should either import and use mpp_error or make a call to MPI_Finalize to properly abort execution.
SHiELD_physics/GFS_layer/GFS_typedefs.F90
Lines 2111 to 2112 in 6a6e94f
From bensonr: The designed way to get this information to the physics from outside is to use the GFS_init_type. This type is populated with dycore and other external to the physics information within the atmos_model.F90::atmos_model_init function and it is passed to GFS_Initialize, where it is passed to the Model%init procedure.
From bensonr: Since we no longer need compatibility with ufs as they've embraced CCPP, we
could start using FMS functions and change the write statements to
mpp_error statement. With the appropriate FATAL condition, mpi will be
properly terminated and any future coupled components running concurrently
will also get the termination signal.
To Reproduce
N/A
Expected behavior
N/A
System Environment
N/A
Additional context
N/A
Hi, @laurenchilutti. I presume we can just do a 'use mpp_mod' within the SHiELD Physics routines so we can call mpp_error()? Or do we need to change how the SHiELD Physics routines are built to bring in the FMS libraries?
@lharris4 - yes, a simple use mpp_mod or even the more inclusive use FMS , which includes every public interface and parameter available from the FMS library.
Hi, @laurenchilutti. I presume we can just do a 'use mpp_mod' within the SHiELD Physics routines so we can call mpp_error()? Or do we need to change how the SHiELD Physics routines are built to bring in the FMS libraries?