DHI / DHI/modelskill

Replace isinstance chains with match/case for observation dispatch

Open
#610 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
56
Forks
9
Avg merge
57m
Merged PRs (30d)
3

Description

`matching.py` already uses match/case for observation type dispatch, but several other files still use isinstance chains:

- `model/dfsu.py` — `extract()` method
- `model/grid.py` — `extract()` method
- `model/dummy.py` — `extract()` method
- `plotting/_spatial_overview.py` — observation type for plotting

These should be converted to match/case for consistency and readability.

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.