Refactor `dodola regrid --cyclic` into new `dodola add-cyclic`

ClimateImpactLab / dodola

Containerized application for running individual tasks in a larger, orchestrated CMIP6 bias-adjustment and downscaling workflow.

https://climateimpactlab.github.io/dodola/

Apache License 2.0

14 stars 7 forks source link

Refactor `dodola regrid --cyclic` into new `dodola add-cyclic` #110

Open brews opened 3 years ago

brews commented 3 years ago

The dodola regrid --cyclic flag isn't really useful in the context of regridding because it usually requires large-scale rechunking of data before regridding. This makes the --cyclic flag useless for our purpose.

A solution to this is to put the logic for --cyclic into it's own service/CLI command.

brews commented 3 years ago

On second thought, we have use for the "cyclic" option in the core functions when applied to a distributed regridding...

Edit: Ahhh but it still fails because adding cyclic adds chunks along the lat/lon dims...

brews commented 2 years ago

I'm not going to have time to handle this next week.