Skip to content

Feature/wdboggs/#4292 test verticalgrid aspect#4525

Open
darianboggs wants to merge 12 commits intorelease/MAPL-v3from
feature/wdboggs/#4292_test_verticalgrid_aspect
Open

Feature/wdboggs/#4292 test verticalgrid aspect#4525
darianboggs wants to merge 12 commits intorelease/MAPL-v3from
feature/wdboggs/#4292_test_verticalgrid_aspect

Conversation

@darianboggs
Copy link
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

This PR implements tests for the update_payload and update_from_payload methods for the VerticalGridAspect type.

Related Issue

#4292

@darianboggs darianboggs requested a review from tclune March 13, 2026 21:57
@darianboggs darianboggs self-assigned this Mar 13, 2026
@darianboggs darianboggs requested a review from a team as a code owner March 13, 2026 21:57
@darianboggs darianboggs 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 labels Mar 13, 2026
@darianboggs
Copy link
Contributor Author

Because of the complexity with VerticalGrid, VerticalStaggerLoc, VerticalAlignment variations (each present or not present), this PR focuses on VerticalGrid. The other variations can be implemented in a subsequent PR.

tclune
tclune previously approved these changes Mar 16, 2026
@darianboggs darianboggs force-pushed the feature/wdboggs/#4292_test_verticalgrid_aspect branch from 2aaf6aa to f4e112d Compare March 18, 2026 14:27
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 🎁 New Feature This is a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants