Skip to content

refactor(utils): allow optional errmsg in handle_allocate_error - #1671

Open
johnpaulalex wants to merge 1 commit into
ESCOMP:cam_developmentfrom
johnpaulalex:fix-issue-1322-errmsg
Open

johnpaulalex wants to merge 1 commit into
ESCOMP:cam_developmentfrom
johnpaulalex:fix-issue-1322-errmsg

Conversation

@johnpaulalex

@johnpaulalex johnpaulalex commented Sep 5, 2026

Copy link
Copy Markdown

Description of Changes

Add optional errmsg param to handle_allocate_error in src/utils/cam_abortutils.F90

Mirrors the fix in ESCOMP/CAM-SIMA#311.

Linked Issues

Fixes ESCOMP/CAM#1322

Description of Generative AI Usage

Google Antigravity was used to write the code and tests, followed by human-guided verification.

Answer-Changing & Restart Compatibility

  • Answer Impact: Bit-for-Bit (B4B). Adds a new optional parameter that nobody uses yet, and which defaults to a no-op.
  • Restart Space: No restart file or state variable impact.

Verification Performed

  • AI built some adhoc tests to verify behavior with and without errmsg. Felt a little heavyweight to submit but they do pass.

@cacraigucar

Copy link
Copy Markdown
Collaborator

@johnpaulalex - Please let us know when this is ready for review by removing it from draft status

@cacraigucar cacraigucar added the misc tag issue or PR candidate for upcoming misc tag label Sep 8, 2026
@johnpaulalex
johnpaulalex marked this pull request as ready for review September 8, 2026 22:47
@johnpaulalex

Copy link
Copy Markdown
Author

No longer draft, thanks @cacraigucar !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

misc tag issue or PR candidate for upcoming misc tag

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants