ESCOMP / ESCOMP/CAM-SIMA

The generated "allocate_physics_types_fields" subroutine should use "check_allocate"

Open
#199 0 comments 0 reactions 0 assignees View on GitHub
enhancement misc
Dominant language
No language data
Stars
9
Forks
22
Avg merge
8d 5h
Merged PRs (30d)
4

Description

Currently the registry generator creates a `allocate_physics_types_fields` subroutine that allocates all of the variables that are being used from the registry. However, there are currently no internal checks as to whether the allocations are actually successful, unlike the rest of the model.

So, the registry generator should be modified so that a `stat` output is added to each `allocate` call, with the resulting error code then checked using the `check_allocate` subroutine.

Contributor guide

No contributing guide indexed for this repository

Research direction

Find the registry generator entry point that emits the allocate_physics_types_fields subroutine, then inspect how allocation calls and existing check_allocate usage are generated. Update the generated allocations to include status handling and checks, regenerate the output, and verify that each allocation is checked.

Written by the indexing model from the issue text.

Assessment

Tech stack
fortran
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.