MetOffice / MetOffice/CSET

Simplify VER_METHOD in PointStat and EnsembleStat (point)

Open
#2,260 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
33
Forks
19
Avg merge
1d 22h
Merged PRs (30d)
30

Description

`VER_METHOD` is likely to be redundant, this issue is to explore if it can be removed.

The separation of both methods is inherited for the VER system. With VER, there were two verification strategies:

- Station Based Verification: Matches forecast-observations pairs on station, and validity time, storing the pairs and using VerPy to calculate and plot the statistics. Allows masking by lists of stations.
- Area Based Verification: Selects the observations on area masks and calculates the statistics for those areas, with VerPy used for plotting.

With METPlus, the method is implicit in the output type. It is likely, then, that `VER_METHOD` is not needed. This issue is to explore whether that is possible.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing VER_METHOD through PointStat and EnsembleStat, then compare how METPlus output types distinguish station-based and area-based verification. Determine whether the setting is redundant without changing either behavior; done means reaching and recording a clear decision about removal and identifying all affected references.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.