Thanks for sharing this cool package! Looks vey useful. Definitely something we can list in "related projects". 😄

We have a concern: This package directly adds some methods to xarray.DataArray by using the accessor feature.
For example, dataarray.isbetween() is implemented by registering an accessor whose name is 'isbetween'.
This usage may not be what is expected by the xarray's developers.

Indeed, we should probably discourage this sort of "accessor hacking". Just out of curiosity, why did you opt for this rather than the more standard, dataarray.compare.isbetween()? I see your package was just released. Are you firmly committed to your API? Or could you consider deprecating it in favor of the recommended approach in a future release?

test failure is unrelated. Thanks @astropenguin .