openradar / xradar

A tool to work in weather radar data in xarray
https://docs.openradarscience.org/projects/xradar
MIT License
88 stars 17 forks source link

MNT: fetch radar data from open-radar-data, add description/url #44

Closed kmuehlbauer closed 1 year ago

kmuehlbauer commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Merging #44 (a0c191b) into main (28ab806) will increase coverage by 0.00%. The diff coverage is 81.25%.

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   85.49%   85.49%           
=======================================
  Files          15       15           
  Lines        3005     3020   +15     
=======================================
+ Hits         2569     2582   +13     
- Misses        436      438    +2     
Flag Coverage Δ
unittests 85.49% <81.25%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
xradar/accessors.py 39.13% <25.00%> (-0.87%) :arrow_down:
xradar/io/backends/cfradial1.py 82.22% <100.00%> (+0.40%) :arrow_up:
xradar/io/backends/furuno.py 90.27% <100.00%> (+0.05%) :arrow_up:
xradar/io/backends/gamic.py 86.61% <100.00%> (+0.10%) :arrow_up:
xradar/io/backends/iris.py 81.61% <100.00%> (+0.03%) :arrow_up:
xradar/io/backends/odim.py 87.98% <100.00%> (+0.05%) :arrow_up:
xradar/io/backends/rainbow.py 82.80% <100.00%> (+0.08%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

kmuehlbauer commented 1 year ago

@mgrover1 Just some maintenance on code and CI.

mgrover1 commented 1 year ago

@kmuehlbauer I think we can go ahead and merge. For some reason code coverage is dropping, but still above 80% which is good!

kmuehlbauer commented 1 year ago

@kmuehlbauer I think we can go ahead and merge. For some reason code coverage is dropping, but still above 80% which is good!

There are more lines removed which counted to coverage before than added, so the overall coverage percentage decreased a bit.