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

EMI_DataMod.F90 syntax error with IBM XL compiler #2976

Closed
sarats opened this issue Jun 6, 2019 · 0 comments · Fixed by #3103
Closed

EMI_DataMod.F90 syntax error with IBM XL compiler #2976

sarats opened this issue Jun 6, 2019 · 0 comments · Fixed by #3103
Assignees

Comments

@sarats
Copy link
Member

sarats commented Jun 6, 2019

XL seems to have an issue with the following line. PGI works fine.

Fix reported in #2985

172 "E3SM/components/clm/src/external_models/emi/src/emi_data_definitio/EMI_DataMod.F90", line 885.22: 1515-019 (S) Syntax is incorrect.
173 ** emi_datamod   === End of Compilation 1 ===
174 1501-511  Compilation failed for file EMI_DataMod.F90.
jgfouca added a commit that referenced this issue Jun 25, 2019
Nuopc cmeps

Bringing the latest nuopc development to master

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

Update gh-pages html (Y/N)?:

Code review:
bishtgautam added a commit that referenced this issue Aug 1, 2019
Removes a comma after write statement

Fixes #2976
Fixes #2985

[BFB]
jqyin added a commit that referenced this issue Aug 1, 2019
Minor fix to ELM for IBM compiler

Removes a comma after write statement

Fixes #2976
Fixes #2985

[BFB]
jqyin added a commit that referenced this issue Aug 2, 2019
Minor fix to ELM for IBM compiler

Removes a comma after write statement

Fixes #2976
Fixes #2985

[BFB]
tanzeli1982 pushed a commit that referenced this issue Aug 12, 2019
Removes a comma after write statement

Fixes #2976
Fixes #2985

[BFB]
rljacob pushed a commit that referenced this issue May 6, 2021
Nuopc cmeps

Bringing the latest nuopc development to master

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

Update gh-pages html (Y/N)?:

Code review:
jgfouca pushed a commit that referenced this issue Oct 9, 2024
…elds_rebase_var

EAMXX uses a different implementation of the atmosphere model, compared to EAM, and moab driver needs to be extended to support a coupled case

case tested is --compset F2010-SCREAMv1-MPASSI --res ne4pg2_oQU480 --driver moab
this case uses mpas sea ice, CICE is not yet supported by MOAB

the implementation follows similar strategy compared to the rest of the models
Atmosphere mesh in moab has 4 different instances, similar to EAM,

corresponding to coarse spectral mesh ( size 6nene),
fine quad mesh (corresponding to GLL quad mesh, size 6nene* (np-1)* (np-1) )
pg2 FV type mesh (size 6nene* pg2 * pg2)
point cloud corresponding to mct simplified vertex grid.

On the coupler side, we still use just the pg2 FV type mesh, used for intersection and coupling with other models
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants