CliMA / CliMA/ClimaCore.jl

ClimaCoreTempestRemap parameter checking

Open
#1,036 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Julia
Stars
117
Forks
19
Avg merge
3d 4h
Merged PRs (30d)
41

Description

**Is your feature request related to a problem? Please describe.**
Certain parameters cause TempestRemap to fail with an undescriptive error, which makes debugging very difficult. This seems to often be caused by constructing a topology with unsupported values of `ne` or `Nq`, which then gets passed into [write_exodus](https://github.com/CliMA/ClimaCore.jl/blob/main/lib/ClimaCoreTempestRemap/src/exodus.jl#L20) and fails.

**Describe the solution you'd like**
We should add assertions to the TR wrappers to prevent the code from reaching TR itself with an invalid topology, and to provide descriptive error messages when failures occur.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the TR wrappers that construct topologies and the write_exodus entry point in lib/ClimaCoreTempestRemap/src/exodus.jl. Trace how ne and Nq reach TempestRemap, then add validation and descriptive failures; done means invalid topologies are rejected before TR and errors explain the bad parameters.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
hpc
Issue type
Feature
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.