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

MOAB api change in projection #6523

Merged
merged 4 commits into from
Aug 16, 2024
Merged

MOAB api change in projection #6523

merged 4 commits into from
Aug 16, 2024

Conversation

iulian787
Copy link
Contributor

@iulian787 iulian787 commented Jul 23, 2024

Add an extra argument, related to CAAS filtering, in iMOAB mapping.

[BFB]


will need to update all MOAB installations when this is merged @vijaysm @rljacob

@rljacob rljacob self-assigned this Jul 23, 2024
Copy link

github-actions bot commented Jul 23, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6523/
on branch gh-pages at 2024-07-25 03:14 UTC

@rljacob rljacob requested a review from vijaysm July 23, 2024 18:30
Copy link
Contributor

@vijaysm vijaysm left a comment

Choose a reason for hiding this comment

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

This is a straightforward change to conform to the new API. If the builds pass, this branch can be merged.

@rljacob rljacob added the BFB PR leaves answers BFB label Aug 16, 2024
rljacob added a commit that referenced this pull request Aug 16, 2024
Add an extra argument, related to CAAS filtering, in iMOAB mapping.

[BFB]
@rljacob rljacob merged commit 5d44648 into master Aug 16, 2024
21 checks passed
@rljacob rljacob deleted the iulian787/moab_api_change branch August 16, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB Coupler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants