MeteoSwiss / MeteoSwiss/dvas

Correct zone flagging to include NaN altitudes at start/end

Open
#267 1 comment 0 reactions 1 assignee Claimed by @fpavogt View on GitHub
wontfix
Dominant language
Python
Stars
4
Forks
0
PR merge metrics
No merged PRs in 30d

Description

The current zone flagging compares the altitude to specific limits, which implies that altitudes with NaNs do not get flagged as anything. This implies that specific points get left out from statistics that users may compile down the line.

It would be better to rely on the index of points to decide which zone they belong to.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.