Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
glemieux opened this issue Sep 25, 2023 · 1 comment
Open

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

glemieux opened this issue Sep 25, 2023 · 1 comment
Assignees
@glemieux
Labels
bug - software engineering

Comments

@glemieux
Copy link
Contributor

glemieux commented Sep 25, 2023 •
edited
Loading

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 glemieux self-assigned this Sep 25, 2023
@glemieux
Copy link
Contributor Author

Note that ESCOMP/CTSM#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.

This was referenced Sep 28, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@glemieux glemieux

Labels
bug - software engineering
Projects
FATES issue board
Status: ❕Todo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@glemieux