`lsm2mask` should be handled better

georgemilosh / Climate-Learning

How to predict extreme events in climate using rare event algorithms and modern tools of machine learning

https://georgemilosh.github.io/Climate-Learning/

MIT License

20 stars 5 forks source link

`lsm2mask` should be handled better #81

Open AlessandroLovo opened 10 months ago

AlessandroLovo commented 10 months ago

In the current implementation, ERA_Fields_New.Plasim_Field.set_mask uses the parameter lsm2mask decide whether to mask only over land. However, with xarray features right now the mask is already only over land.

This is technically not a bug, but it may make things confusing