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.

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

Satellite phenology mode run fails with gnu compiler #832

Closed
glemieux opened this issue Jan 28, 2022 · 1 comment · Fixed by #838
Closed

Satellite phenology mode run fails with gnu compiler #832

glemieux opened this issue Jan 28, 2022 · 1 comment · Fixed by #838

Comments

@glemieux
Copy link
Contributor

glemieux commented Jan 28, 2022

Additional testing of satellite phenology mode with E3SM using the gnu compiler resulted in the follow issue: E3SM-Project/E3SM#4755. SP mode hadn't been tested with the gnu compiler as both our clm and elm regression tests for this mode utilize the intel compiler.

Replicating the issue on Cori the results in the following error immediately during initialization:

378:  in sub-canopy layer in leafc_from_treelai
378:  this is not set up to work for lower canopy layers.
378:  problem in leafc_from_treelai         379           2
378:  ENDRUN:ERROR in /global/u1/g/glemieux/E3SM/components/elm/src/external_models/fates/biogeochem/FatesAllometryMod.F90 at line 811                                                                                                                                                                                                                                                                                                                                                       

Tracing the code, I determined that the issue has to do with the cohort%canopy_layer being un-initialized and not specific to elm-fates. This is pretty clear from the error message above as the 379 is the canopy_layer trying to be indexed here, which doesn't make sense for fates generally. I confirmed this with a test run on Cheyenne as well. I've implemented a simple fix to assign the temporary starting canopy_layer value to be one, but this has exposed a couple other issues with sp mode initialization as well that need to be addressed.

Note: sp mode will run with the gnu compiler if the case or test is set up in debug mode.

@rosiealice
Copy link
Contributor

nice job @glemieux :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants