ARM-DOE / pyart

The Python-ARM Radar Toolkit. A data model driven interactive toolkit for working with weather radar data.
https://arm-doe.github.io/pyart/
Other
516 stars 268 forks source link

ADD: `radar_freq` param in `hydroclass_semisupervised` #1573

Closed RBhupi closed 5 months ago

RBhupi commented 6 months ago
  1. Added radar_freq option when frequency information is missing from the radar object.
  2. Now always print warning showing frequency used by the function.
  3. Added docstring for thechange.
  4. Modified the old notes section to include radar_freq.
review-notebook-app[bot] commented 6 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

RBhupi commented 6 months ago

This is failing because of the missing file. I want to use this file in my example which I can take from the ARM servers.

WARNING: ../../examples/retrieve/plot_hydrometeor_class_x_band.py unexpectedly failed to execute correctly:

Traceback (most recent call last):
  File "/home/runner/work/pyart/pyart/examples/retrieve/plot_hydrometeor_class_x_band.py", line 29, in <module>
    radar = pyart.io.read(files[0])
                          ~~~~~^^^
IndexError: list index out of range
zssherman commented 6 months ago

@RBhupi If you want to pull the file and run that, you can do a PR adding the file to https://github.com/openradar/open-radar-data

then once the file has been added to the data directory, run python make_registry.py

zssherman commented 5 months ago

@RBhupi This looks good to me now!