search

barronh / pseudonetcdf

PseudoNetCDF like NetCDF except for many scientific format backends
GNU Lesser General Public License v3.0
76 stars 35 forks source link

windrose #113

Closed zfarooqui closed 2 years ago

zfarooqui commented 4 years ago

HI Barron,

I created pollution rose script using windrose from Pseudonetcdf and found that windrose is not classifying the bins correctly.

Note: args.fromnorth

https://github.com/barronh/pseudonetcdf/blob/master/scripts/pncwindrose.py Line 42: thetas = ((np.radians((wd) // width * width)).astype('f') %

needs to be

thetas = ((np.radians((wd + halfwidth) // width * width)).astype('f') %

I request you to please take out some time and look into the issues. If you need data/binning/plot I created then I can provide.

Thank you,

barronh commented 3 years ago

Thanks for posting this. Sorry it is taking so long for me to update.

barronh commented 2 years ago

Addressed in main. I have quit using the scripts all-together, so this fell off my radar. Sorry @zfarooqui