issues

• [x] Closes https://github.com/pydata/xarray/issues/7971
• [x] Closes https://github.com/pydata/xarray/issues/8437.

We were previously pinned to <0.21 Removing the pin didn't change the env but with >=0.21 we get 0.22 which works.

• [ ] Tests added - check bottleneck tests
• [ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

• [x] Closes #7841
• [x] Closes #7892

• [x] Closes #7797
• [x] Tests added
• [x] User visible changes (including notable bug fixes) are documented in whats-new.rst

cc @slevang

Pandas is expecting to release v2 in two weeks (pandas-dev/pandas#46776 (comment)). But we are still incompatible with their main branch: - #7441 - #7420

This PR pins pandas to <2

There was a bug in estimating the last index of the slice. Index a range object instead.

• [x] Closes #6560
• [x] Tests added

Oops. I used "Github Codespaces" to edit whats-new, but it turns if you commit in there, it just commits to main!

This fixes the pre-commit error.

• [x] Closes #3141, Closes #3417
• [x] Tests added
• [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
• [x] New functions/methods are listed in api.rst

• xref #5409
• [x] Tests added
• [x] Passes pre-commit run --all-files

Some questions: 1. flake8 fails with some false positives. What do we do about that? 2. I am importing the da and ds fitures from test_dataarray and test_dataset. Is that the pattern we want to follow?

xarray/tests/test_coarsen.py:9:1: F401 '.test_dataarray.da' imported but unused xarray/tests/test_coarsen.py:10:1: F401 '.test_dataset.ds' imported but unused xarray/tests/test_coarsen.py:13:36: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:20:26: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:35:25: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:51:5: F811 redefinition of unused 'da' from line 9 xarray/tests/test_coarsen.py:62:5: F811 redefinition of unused 'da' from line 9 xarray/tests/test_coarsen.py:84:5: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:156:5: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:186:25: F811 redefinition of unused 'ds' from line 10 xarray/tests/test_coarsen.py:217:5: F811 redefinition of unused 'da' from line 9 xarray/tests/test_coarsen.py:269:5: F811 redefinition of unused 'da' from line 9

• [x] Closes #4373
• [x] Tests added
• [x] Passes isort . && black . && mypy . && flake8
• [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
• [x] New functions/methods are listed in api.rst

I could use some help with adding tests and parameter checking if someone wants to help :)

``` python import numpy as np import xarray as xr

ds = xr.Dataset() ds.coords["x"] = ("x", np.arange(10)) ds.coords["y"] = ("y", np.arange(20)) ds.coords["t"] = ("t", np.arange(4)) ds["u"] = np.sin((ds.x - 5) / 5) * np.sin((ds.y - 10) / 10) ds["v"] = np.sin((ds.x - 5) / 5) * np.cos((ds.y - 10) / 10)

ds = ds * 2*np.cos((ds.t) * 2 * 3.14 /0.75)

ds["u"].attrs["units"] = "m/s" ds["mag"] = np.hypot(ds.u, ds.v)

ds.mag.plot(col="t", x="x")

fg = ds.plot.quiver(x="x", y="y", u="u", v="v", col="t", hue="mag") ```

• [x] Closes #4019
• [x] Tests added
• [x] Passes isort -rc . && black . && mypy . && flake8
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

~Doesn't have a proper test. Need some help here~. cc @bnaul

• [x] Closes #3569
• [x] Tests added
• [x] Passes isort -rc . && black . && mypy . && flake8
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

Not sure how to test this.

• [x] Closes #3681
• [x] Tests added
• [x] Passes isort -rc . && black . && mypy . && flake8
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

I added an example on using apply_ufunc to vectorize a 1D example over a DataArray. Comments and feedback welcome.

I added an example of using numba's guvectorize too. Thoughts on keeping that bit?

• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

cc @rabernat @TomNicholas

Dask arrays with the same graph have the same name. We can use this to quickly compare dask-backed variables without computing. (see https://github.com/pydata/xarray/issues/3068#issuecomment-508853564)

I will work on adding extra tests but could use feedback on the approach.

• [x] Closes #3068, closes #3311, closes #3454
• [x] Tests added
• [x] Passes black . && mypy . && flake8
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

@djhoese, thanks for the great example code!

• [x] Closes #508
• [x] Tests added
• [x] Passes black . && mypy . && flake8
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

Downstream issue: https://github.com/marbl-ecosys/cesm2-marbl/issues/1

Basically, we are currently raising an error when attempting to merge variables that are present in some datasets but not others that are provided to concat. This seems unnecessarily strict and it turns out we had an issue for it! (#508)

This fixes the following behaviour. (downstream issue https://github.com/xgcm/xgcm/issues/154)

def test_concat(self, data): split_data = [ data.isel(dim1=slice(3)), data.isel(dim1=3), # this wouldn't work on master data.isel(dim1=slice(4, None)), ] assert_identical(data, concat(split_data, "dim1"))

• partially addresses #2909 , closes #2901, closes #2908

Unfortunately my fix in #2935 broke some major functionality. A proper fix would involve some facetgrid refactoring I think; so that'll take some time. This reverts that commit and adds a test.

xr.DataArray(np.random.randn(10, 20)).plot(levels=10, cmap=mpl.cm.RdBu)

• [x] Closes #1354
• [x] Tests added
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

I won't work on it for the next few days if someone else wants to take over...

• [x] Tests added (for all bug fixes or enhancements)
• [x] Tests passed (for all non-documentation changes)
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

Adds a labels boolean kwarg for contour that adds contour labels. Also adds clabel_kwargs that is passed on to Axes.clabel()

air.isel(time=0).plot.contour(labels=True, colors='k', clabel_kwargs={'fmt': '%.1f'})

• [x] Closes #2583
• [x] Tests added (for all bug fixes or enhancements)
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

• [x] Closes #2381
• [x] Tests added (for all bug fixes or enhancements)
• [x] Tests passed (for all non-documentation changes)
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

• [x] Tests passed (for all non-documentation changes)

Remove some warnings.

• [x] Closes #2224 (remove if there is no corresponding issue, which should only be the case for minor changes)
• [x] Tests added (for all bug fixes or enhancements)
• [x] Tests passed (for all non-documentation changes)
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

This PR adds support for xscale, yscale, xticks, yticks, xlim, ylim kwargs following discussion in #2224 and the Pandas API (https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.plot.html#pandas.DataFrame.plot).

Haven't added FacetGrid support yet. I'll get to that soon.

• [x] Closes #2284
• [x] Tests added (for all bug fixes or enhancements)
• [x] Tests passed (for all non-documentation changes)
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

```python da = xr.DataArray([1, 5], dims=['time'], coords={'time': [np.datetime64('2014-05-06'), np.datetime64('2014-05-10')]})

da.interp(time='2014-05-07') <xarray.DataArray ()> array(2.) Coordinates: time datetime64[ns] 2014-05-07 ```

• [ ] Closes #xxxx
• [x] Tests added / passed
• [x] Passes git diff upstream/master **/*py | flake8 --diff
• [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

This PR teaches plot.contourf() to contour variables with both a 1D co-ordinate (e.g. time) and a 2D co-ordinate (e.g. time-varying depth) da: <xarray.DataArray 'S' (z: 5, time: 85646)> array([[ nan, nan, nan, ..., nan, nan, nan], [ 35.02816 , 34.729567, 34.779223, ..., 34.57513 , 34.671975, 34.334675], [ 35.206943, 35.163239, 34.938674, ..., 34.780728, 34.836331, 34.70386 ], [ nan, 35.184057, 35.10592 , ..., 34.656925, 34.776915, 34.429442], [ 34.85562 , 34.81994 , 35.00963 , ..., 35.014522, 34.9747 , 35.033848]]) Coordinates: depth (z, time) float16 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 0.0 53.5 ... * time (time) datetime64[ns] 2013-12-19T06:00:01 2013-12-19T06:10:01 ... Dimensions without coordinates: z

Now we can do da.plot(x='time', y='depth')

Couple of questions: 1. I've added a test, but am not sure how to work in an assert statement. 2. How do I test that I haven't messed up the syntax in whats-new.rst