Skip to content

Added support to fix a memory leak in mkIAU#4333

Merged
atrayano merged 7 commits intodevelopfrom
bugfix/atrayano/memleakfix
Jan 26, 2026
Merged

Added support to fix a memory leak in mkIAU#4333
atrayano merged 7 commits intodevelopfrom
bugfix/atrayano/memleakfix

Conversation

@atrayano
Copy link
Contributor

@atrayano atrayano commented Jan 26, 2026

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 is needed to fix a memory leak in mkIAU

Related Issue

@atrayano atrayano self-assigned this Jan 26, 2026
@atrayano atrayano added the 🪲 Bugfix This fixes a bug! label Jan 26, 2026
@atrayano atrayano requested a review from a team as a code owner January 26, 2026 16:56
@atrayano atrayano added the 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. label Jan 26, 2026
Copy link
Collaborator

@tclune tclune left a comment

Choose a reason for hiding this comment

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

See suggested fix for log message and a question about MAX_FORMATTERS

Copy link
Collaborator

@tclune tclune left a comment

Choose a reason for hiding this comment

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

I've reverted MAX_FORMATTERS. In a separate ticket we can explore the consequences of changing the value.

@atrayano atrayano merged commit 09344ff into develop Jan 26, 2026
40 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. 🪲 Bugfix This fixes a bug!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants