search

NGEET / fates

repository for the Functionally Assembled Terrestrial Ecosystem Simulator (FATES)
Other
105 stars 92 forks source link

Cross-grid seed dispersal mechanism is not b4b for PE layout changes #1089

Open glemieux opened 1 year ago

glemieux commented 1 year ago

During testing of #1005, I discovered that the seed dispersal mechanism is not b4b between different layout changes (i.e the test runs fail ERP or PEM tests). This is likely due to order of operation issues in collecting floating point values with the MPI collective routines. Note that ERS tests pass COMPARE_BASE_REST checks.

glemieux commented 1 year ago

Note that https://github.com/ESCOMP/CTSM/pull/2077 adds a cross-grid seed dispersal testmod that is called with a layout changing PEM testtype option. This will further help us track and address the issue in the future.