Skip to content

Disdronator#128

Merged
siiptuo merged 7 commits intomainfrom
disdronator
Mar 11, 2026
Merged

Disdronator#128
siiptuo merged 7 commits intomainfrom
disdronator

Conversation

@siiptuo
Copy link
Contributor

@siiptuo siiptuo commented Mar 11, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds initial support for Disdromet RD-80 disdrometer processing by introducing a new RD-80 Level 1b converter and a new internal disdronator reader package, plus a small plotting tweak and a unit test + sample file.

Changes:

  • Add rd802nc converter for Disdromet RD-80 and register it in cloudnetpy.instruments.
  • Introduce new cloudnetpy.disdronator readers (RD-80 + Parsivel + LPM) and shared parsing utilities.
  • Update plotting metadata to mask zeros for number_concentration, and extend mypy overrides for cftime.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tests/unit/test_disdrometer.py Adds a unit test for the new RD-80 converter output.
tests/unit/data/rd80/RD-211231-181400.txt Adds a small RD-80 sample input file for tests.
pyproject.toml Ignores missing type stubs for cftime.* in mypy.
cloudnetpy/plotting/plot_meta.py Masks zeros when plotting number_concentration.
cloudnetpy/instruments/instruments.py Registers the RD-80 instrument metadata.
cloudnetpy/instruments/disdrometer/rd80.py Implements RD-80 → Cloudnet Level 1b conversion.
cloudnetpy/instruments/disdrometer/init.py Exposes rd802nc from the disdrometer subpackage.
cloudnetpy/instruments/init.py Re-exports rd802nc alongside other instrument converters.
cloudnetpy/disdronator/utils.py Adds shared conversion helper for parsing outputs into numpy arrays.
cloudnetpy/disdronator/rd80.py Adds RD-80 ASCII reader + bin constants.
cloudnetpy/disdronator/parsivel.py Adds Parsivel reader supporting multiple formats.
cloudnetpy/disdronator/lpm.py Adds LPM reader for multiple formats.
cloudnetpy/disdronator/init.py Exposes reader entry points from the new package.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@siiptuo siiptuo merged commit 0fa8c68 into main Mar 11, 2026
14 checks passed
@siiptuo siiptuo deleted the disdronator branch March 11, 2026 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants