Skip to content

Feature/wjiang/destroy mgridded#4126

Merged
mathomp4 merged 7 commits intodevelopfrom
feature/wjiang/destroy_mgridded
Oct 15, 2025
Merged

Feature/wjiang/destroy mgridded#4126
mathomp4 merged 7 commits intodevelopfrom
feature/wjiang/destroy_mgridded

Conversation

@weiyuan-jiang
Copy link
Contributor

@weiyuan-jiang weiyuan-jiang commented Oct 14, 2025

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

Fixed the instantaneous history output and release the memory in TileGridIO

Related Issue

@weiyuan-jiang weiyuan-jiang requested a review from a team as a code owner October 14, 2025 18:15
@weiyuan-jiang weiyuan-jiang added 🪲 Bugfix This fixes a bug! 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. labels Oct 14, 2025
@mathomp4 mathomp4 changed the base branch from main to develop October 14, 2025 18:21
@mathomp4 mathomp4 added the 😖 Shutdown This occurred during shutdown label Oct 14, 2025
mathomp4
mathomp4 approved these changes Oct 15, 2025
View reviewed changes
Copy link
Member

@mathomp4 mathomp4 left a comment

Choose a reason for hiding this comment

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

Zero-diff in my testing.

@mathomp4 mathomp4 merged commit e6aef9d into develop Oct 15, 2025
39 checks passed
@mathomp4 mathomp4 deleted the feature/wjiang/destroy_mgridded branch October 15, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mathomp4 mathomp4 mathomp4 approved these changes

Assignees

No one assigned

Labels

0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 🪲 Bugfix This fixes a bug! 😖 Shutdown This occurred during shutdown

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@weiyuan-jiang @mathomp4