search

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

Update ppi example to use new accessor #61

Closed mgrover1 closed 1 year ago

mgrover1 commented 1 year ago

This updates the PPI example to use the new georeference accessor

codecov[bot] commented 1 year ago

Codecov Report

Merging #61 (8ec866a) into main (f74952a) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   83.43%   83.43%           
=======================================
  Files          17       17           
  Lines        3146     3146           
=======================================
  Hits         2625     2625           
  Misses        521      521
Flag Coverage Δ
unittests 83.43% <ø> (ø)

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

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 Not sure what's going on with the one breaking notebook. Either a real problem with the new accessor or a fluke.

mgrover1 commented 1 year ago

@mgrover1 Not sure what's going on with the one breaking notebook. Either a real problem with the new accessor or a fluke.

I reran that test to see if it was a fluke... also removed that last blank cell. We shall see!