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

Fixes #3075 - propagate time-varying aspects of fields #3095

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

tclune
Copy link
Collaborator

@tclune tclune commented Oct 13, 2024

Required updating various things to use new MAPL_Info interfaces.

Need to be careful that keys sent to ESMF do not end in "/" and yet we don't want users to have to prepend their keys with "/". Fix for now is to check and prepend with "/" if not present.

If performance is found to be inadequate the issue will be revisited.

Types of change(s)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Trivial change (affects only documentation or cleanup)
  • Refactor (no functional changes, no api changes)

Checklist

  • Tested this change with a run of GEOSgcm
  • Ran the Unit Tests (make tests)

Description

Related Issue

Required updating various things to use new MAPL_Info interfaces.

Need to be careful that keys sent to ESMF do not end in "/" and yet we
don't want users to have to prepend their keys with "/".  Fix for now
is to check and prepend with "/" if not present.

If performance is found to be inadequate the issue will be revisited.
@tclune tclune added 🎁 New Feature This is a new feature 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 📈 MAPL3 MAPL 3 Related Changelog Skip Skips the Changelog Enforcer labels Oct 13, 2024
@tclune tclune requested a review from a team as a code owner October 13, 2024 17:50
@mathomp4
Copy link
Member

CI is weird. Seems like a Github problem. I'll re-run

@tclune
Copy link
Collaborator Author

tclune commented Oct 14, 2024

CI is weird. Seems like a Github problem. I'll re-run

Probably not the issue. I keep forgetting to run the cap tests that also exercise the generic layer. So my bad for pushing before all tests at least pass in my development environment.

But - now that I want to build on this, I'm going to use SU to get it into the code base.

@tclune tclune merged commit 28408f3 into release/MAPL-v3 Oct 14, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 Diff The changes in this pull request have verified to be zero-diff with the target branch. Changelog Skip Skips the Changelog Enforcer 📈 MAPL3 MAPL 3 Related 🎁 New Feature This is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants