Update handle_allocate_error to include errmsg

Open Beginner friendly
#1,322 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
75/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Quiet
Tech stack
fortran
Domain
hpc

Research direction

Start in cam_abortutils.F90 and compare the requested handling with the referenced CAM-SIMA implementation in pull request 311. Confirm how handle_allocate_error currently builds its error message, then add optional errmsg support so it appears in the overall message. Done means the optional argument is accepted and included without changing behavior when it is absent.

Written by the indexing model from the issue text.

Description

enhancement
What is the feature/what would you like to discuss?

Since the optional errmsg argument to allocate is "new" as of fortran 2003, it'd be nice to include it as an optional argument to handle_allocate_error (in cam_abortutils.F90) to be added to the overall error message.

We can parallel @jimmielin 's implementation in CAM-SIMA - https://github.com/ESCOMP/CAM-SIMA/pull/311

Is there anyone in particular you want to be part of this conversation?

No response

Will this change (regression test) answers?

No

Will you be implementing this enhancement yourself?

Any CAM SE can do this

Dominant language
No language data
Stars
91
Forks
183
Avg merge
3d 10h
Merged PRs (30d)
8

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ESCOMP/CAM

All issues in ESCOMP/CAM

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.