Skip to content

Conversation

@mark-petersen
Copy link
Contributor

@mark-petersen mark-petersen commented Sep 23, 2022

Mesh variables are declared in mpas_ocn_mesh.F rather than individual routines.

Fixes #5200
[BFB]

@mark-petersen mark-petersen added MPAS-ocean Concerning the MPAS-ocean model coupled to E3SM. bug fix PR BFB PR leaves answers BFB labels Sep 23, 2022
@mark-petersen
Copy link
Contributor Author

@knbarton please confirm that this fixes your problem.

Copy link
Contributor

@knbarton knbarton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this and it now runs with scalar SAL and no error.

jonbob added a commit that referenced this pull request Oct 11, 2022
…ext (PR #5201)

Remove unneeded mesh declaration

Mesh variables are declared in mpas_ocn_mesh.F rather than individual
routines.

Fixes #5200
[BFB]
@jonbob
Copy link
Contributor

jonbob commented Oct 11, 2022

passes:

  • ERS_Ld5.T62_oQU120.CMPASO-NYF.chrysalis_intel
  • SMS_D_Ld1.ne30pg2_EC30to60E2r2.WCYCL1850.chrysalis_intel.allactive-wcprod

merged to next

@jonbob jonbob merged commit b02bfec into E3SM-Project:master Oct 12, 2022
@jonbob
Copy link
Contributor

jonbob commented Oct 12, 2022

merged to master

@mark-petersen mark-petersen deleted the mark-petersen/ocean/remove-SAL-mesh-declaration branch October 31, 2022 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BFB PR leaves answers BFB bug fix PR MPAS-ocean Concerning the MPAS-ocean model coupled to E3SM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Segfault when running MPAS-Ocean with serial SAL

3 participants