Use None as default "empty" parameters

Ouranosinc / xclim

Library of derived climate variables, ie climate indicators, based on xarray.

https://xclim.readthedocs.io/en/stable/

Apache License 2.0

333 stars 59 forks source link

Use None as default "empty" parameters #167

Closed huard closed 5 years ago

huard commented 5 years ago

xclim version: master

Description

subset_bbox and subset_gridpoint use '' as a default. Since te expected values are floats, using a string as a default could be confusing for users and maintainers. I suggest using None as the empty default.

huard commented 5 years ago

Fixed by #168