Skip to content

Add l3 readers#87

Open
ghiggi wants to merge 10 commits intomainfrom
add-l3-readers
Open

Add l3 readers#87
ghiggi wants to merge 10 commits intomainfrom
add-l3-readers

Conversation

@ghiggi
Copy link
Copy Markdown
Owner

@ghiggi ghiggi commented Mar 24, 2026

Prework

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Documentation
  • Tutorial
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and communicate accordingly:

The PR fulfills these requirements:

  • It's submitted to the branch named as follow:
    • Fix a bug: bugfix-<some_key>-<word>
    • Improve the doc: doc-<some_key>-<word>
    • Improve a tutorial tutorial-<some_key>-<word>
    • Add a new feature: feature-<some_key>-<word>
    • Refactor some code: refactor-<some_key>-<word>
    • Optimize some code: optimize-<some_key>-<word>
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • Don't forget to link PR to issue if you are solving one.
  • All tests are passing.
  • New/updated tests are included

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Related GitHub issues and pull requests

  • Ref: #

Summary

Please explain the purpose and scope of your contribution.

Copilot AI review requested due to automatic review settings March 24, 2026 12:57
Copy link
Copy Markdown

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 updates core utilities and configuration to support new vertical “top/near-surface” range helpers, expands plotting/colorbar support (notably for xradar + disdrodb), and bumps baseline runtime/dependency requirements.

Changes:

  • Bump minimum supported Python to 3.11 and update core dependency minimum versions.
  • Add new vertical-range helpers (top/near-surface bin + slice utilities) and a height-based range subsetting wrapper, with accompanying tests for the new top/near-surface helpers.
  • Extend xradar plotting with an extent parameter and update/expand colorbar definitions (xradar, default, disdrodb).

Reviewed changes

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

Show a summary per file
File Description
pyproject.toml Updates dependency minimums and raises requires-python to 3.11.
gpm/visualization/plot.py Adds Cartopy gridliner utilities for label removal.
gpm/utils/manipulations.py Adds top/near-surface bin/slice helpers, a subset_range_by_height wrapper, and adjusts infill coordinate handling.
gpm/tests/test_utils/test_manipulations.py Adds fixtures and tests for the new top/near-surface bin and slice helpers.
gpm/io/checks.py Simplifies UTC-time retrieval relying on Python 3.11+ datetime.UTC.
gpm/gv/xradar/methods.py Adjusts georeferencing and adds optional extent handling to plot_map.
gpm/gv/xradar/accessors.py Exposes the new extent argument through the accessor API.
gpm/etc/colorbars/xradar.yaml Refines radar phase/attenuation colorbar definitions and adds variable references.
gpm/etc/colorbars/disdrodb.yaml Adds a new colorbar configuration set for disdrodb/DSD variables.
gpm/etc/colorbars/default_colorbars.yaml Adds a rainfall_rate colorbar referenced by other configs.
gpm/etc/colorbars/2A_DPR_colorbars.yaml Removes Dm/Nw entries (likely moved/reorganized into other configs).
gpm/dataset/decoding/decode_2a_radar.py Adds decoding for precipWater variable.
gpm/accessor/methods.py Exposes new manipulation helpers via the .gpm accessor (top/near-surface + height subsetting).

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

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

❌ Patch coverage is 92.81437% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.83%. Comparing base (fbc988f) to head (aa559de).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
gpm/utils/manipulations.py 89.65% 6 Missing ⚠️
gpm/visualization/plot.py 33.33% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             main      #87    +/-   ##
========================================
  Coverage   90.83%   90.83%            
========================================
  Files         135      135            
  Lines       17537    17703   +166     
========================================
+ Hits        15929    16080   +151     
- Misses       1608     1623    +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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