fmaussion / salem

Add geolocalised subsetting, masking, and plotting operations to xarray
http://salem.readthedocs.io
Other
186 stars 44 forks source link

HAR/TU WRF-Mercator #242

Closed benatouba closed 4 months ago

benatouba commented 5 months ago

Add support for WRF-Mercator projection in TU style products

fmaussion commented 4 months ago

Thanks! Could you make a clean PR without the previous commit? Also this might need some form of testing - but we can think about this later

fmaussion commented 4 months ago

Did you rebase to latest master? I'm surprised the tests are still failing.

benatouba commented 4 months ago

Did the rebase now. Adding a fix for your remark next

benatouba commented 4 months ago

As for testing, i can upload a small test file to the sample data repository

fmaussion commented 4 months ago

Nah thanks, it's ok like this - I'm still puzzled as to why the tests fail, let's check on master