CliMA / CliMA/ClimaAnalysis.jl

Add units check for to_pressure_coordinates

Open
#185 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Julia
Stars
12
Forks
5
Avg merge
6d 17h
Merged PRs (30d)
3

Description

When converting to pressure coordinates without specifying a target pressure, the mapping is done by Plvl(z) = P0 * exp(-z / H_EARTH), where H_EARTH = 7000.0 and P0 = 1e5. The units should be checked, so that the conversion makes sense. For example, if it is kPa instead of Pa.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the implementation of to_pressure_coordinates and inspect how pressure units are represented before the no-target-pressure mapping is applied. Determine the expected behavior for inputs expressed in units such as kPa rather than Pa, then verify that the conversion rejects or handles inconsistent units as intended.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
data
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.