Conversation
|
Is this still a draft or ready for a review? |
|
Sorry, I change it to be ready to review. Thanks. |
| logical :: debug = .false. | ||
| logical :: sync = .false. | ||
| logical :: first_time_step = .false. | ||
| logical :: write_first_time_step = .false. |
There was a problem hiding this comment.
Is there a specific reason for this name change? I would prefer keeping the current name to avoid having to update all of our runscripts.
There was a problem hiding this comment.
I change the name, because I introduced another 'first_time_step' variable in physics here
./GFS_layer/GFS_typedefs.F90: logical :: first_time_step !< flag signaling first time step for time integration routine
This variable is used in the atmos_model.F90
./atmos_model.F90: IPD_control%first_time_step = .false.
./atmos_model.F90: IPD_control%first_time_step = .true.
Also, I think write_first_time_step better indicates the meaning of this variables.
…_offset in SHiELD/atmos_model.F90
…ads into stochastic_physics_wrapper
Description
This is part of the changes to update the use of stochastic physics from a new and updated repo
How Has This Been Tested?
A forecast test was run to compare with the main branch and the results are bit-wise identical.
Checklist: