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

remove seq_flds dependency for mpas_moabmesh #6168

Merged
merged 3 commits into from
Feb 1, 2024

Conversation

iulian787
Copy link
Contributor

@iulian787 iulian787 commented Jan 24, 2024

mpas framework should not depend on driver-moab/shr
mpas ocean cannot be built in standalone mode with MOAB, because of this

[BFB]

mpas framework should not depend on driver/shr
this was a mistake
Copy link

github-actions bot commented Jan 24, 2024

PR Preview Action v1.4.6
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6168/
on branch gh-pages at 2024-01-26 00:16 UTC

@rljacob rljacob requested a review from jonbob January 25, 2024 00:45
@rljacob rljacob self-assigned this Jan 25, 2024
@jonbob
Copy link
Contributor

jonbob commented Jan 25, 2024

@iulian787 -- do we need to do something similar for mpas-albany-landice? Or has moab not been integrated for all components?

@rljacob
Copy link
Member

rljacob commented Jan 25, 2024

We haven't done land ice yet.

@rljacob rljacob requested a review from sbrus89 January 26, 2024 06:19
@rljacob
Copy link
Member

rljacob commented Jan 26, 2024

I mean we have not integrated MOAB to land ice so nothing to do there in this PR.

@sbrus89
Copy link
Contributor

sbrus89 commented Jan 26, 2024

With the extra commit I pushed, I was able to successfully compile standalone MPAS-Ocean on Perlmutter with the dev_compass_1.2.0 compass environment using USE_MOAB=true and MOAB_PATH=/global/cfs/cdirs/e3sm/software/moab/gnu.

@rljacob
Copy link
Member

rljacob commented Jan 29, 2024

@sbrus89 can you approve this?

Copy link
Contributor

@jonbob jonbob left a comment

Choose a reason for hiding this comment

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

Approved based on a quick sanity test and visual inspection

Copy link
Contributor

@sbrus89 sbrus89 left a comment

Choose a reason for hiding this comment

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

Approved based on successful standalone compilation on Perlmutter.

rljacob added a commit that referenced this pull request Jan 30, 2024
Remove mpas framework dependency on driver-moab/shr
mpas ocean cannot be built in standalone mode with MOAB, because of this

[BFB]
@rljacob
Copy link
Member

rljacob commented Jan 30, 2024

merged to next

@rljacob rljacob merged commit 4cc2ced into master Feb 1, 2024
3 checks passed
@rljacob rljacob deleted the iulian787/remove_shr_dependency branch February 1, 2024 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants