Skip to content

Add MAPL_StateMerge to state library#5199

Merged
tclune merged 6 commits into
developfrom
feature/wjiang/state_merge
Jul 3, 2026
Merged

Add MAPL_StateMerge to state library#5199
tclune merged 6 commits into
developfrom
feature/wjiang/state_merge

Conversation

@weiyuan-jiang

Copy link
Copy Markdown
Contributor

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

@weiyuan-jiang weiyuan-jiang requested a review from a team as a code owner June 26, 2026 16:39
@weiyuan-jiang weiyuan-jiang added 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 📈 MAPL3 MAPL 3 Related labels Jun 26, 2026
Weiyuan Jiang added 2 commits June 26, 2026 13:14
ESMF_StateItem_Flag is an opaque derived type with no pFUnit
assertEqual overload. Switch to assertTrue with == comparison.
Comment thread superstructure/state/StateMerge.F90 Outdated
Comment thread superstructure/state/StateMerge.F90 Outdated

@tclune tclune left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I put in some suggestions, but I don't want to obligate you to make those changes. If you want to delay those fixes to a subsequent pass, I can go in and approve for now.

tclune
tclune previously approved these changes Jul 2, 2026
@tclune tclune merged commit cd9dbfc into develop Jul 3, 2026
29 of 32 checks passed
@tclune tclune deleted the feature/wjiang/state_merge branch July 3, 2026 12:57
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. 📈 MAPL3 MAPL 3 Related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants