issues

Table actions

71 rows where type = "pull" and user = 10050469 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: comments, created_at (date), updated_at (date), closed_at (date)

type 1

  • pull · 71 ✖

state 1

repo 1

id node_id number title user state locked assignee milestone comments created_at updated_at ▲ closed_at author_association active_lock_reason draft pull_request body reactions performed_via_github_app state_reason repo type
1224347682 PR_kwDOAMm_X843PYZJ 6569 Add some warnings about rechunking to the docs fmaussion 10050469 closed 0     3 2022-05-03T16:48:02Z 2022-05-10T05:54:13Z 2022-05-10T05:54:05Z MEMBER   0 pydata/xarray/pulls/6569

This adds some warnings at the right places when rechunking a dataset opened with open_mfdataset (see https://github.com/pangeo-data/rechunker/issues/100#issuecomment-1116189019 for context)

Thanks to @dcherian for the wisdom of the day!

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/6569/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1051772149 PR_kwDOAMm_X84ucj1i 5981 Allow string formatting of scalar DataArrays fmaussion 10050469 closed 0     4 2021-11-12T09:44:43Z 2022-05-09T15:25:25Z 2022-05-09T15:25:02Z MEMBER   0 pydata/xarray/pulls/5981

This is a first try at formatting dataarray scalars. Here is the current behavior:

```python In [1]: import xarray as xr ...: import numpy as np

In [2]: a = np.array(1) ...: da = xr.DataArray(a)

In [3]: print(a) 1

In [4]: print(da) <xarray.DataArray ()> array(1)

In [5]: print('{}'.format(a)) 1

In [6]: print('{}'.format(da)) <xarray.DataArray ()> array(1)

In [7]: print('{:.3f}'.format(a)) 1.000

In [8]: print('{:.3f}'.format(da)) 1.000

In [9]: a = np.array([1, 2]) ...: da = xr.DataArray(a)

In [10]: print('{}'.format(a)) [1 2]

In [11]: print('{}'.format(da)) <xarray.DataArray (dim_0: 2)> array([1, 2]) Dimensions without coordinates: dim_0

In [12]: print('{:.3f}'.format(a))

TypeError Traceback (most recent call last) <ipython-input-12-c5afc7863e89> in <module> ----> 1 print('{:.3f}'.format(a))

TypeError: unsupported format string passed to numpy.ndarray.format

In [13]: print('{:.3f}'.format(da))

TypeError Traceback (most recent call last) <ipython-input-13-bddebd8462bd> in <module> ----> 1 print('{:.3f}'.format(da))

~/disk/Dropbox/HomeDocs/git/xarray/xarray/core/common.py in format(self, format_spec) 162 return formatting.array_repr(self) 163 # Else why fall back to numpy --> 164 return self.values.format(format_spec) 165 166 def _iter(self: Any) -> Iterator[Any]:

TypeError: unsupported format string passed to numpy.ndarray.format ``` I don't think there is any backwards compatibility issue but lets see if the tests pass

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5981/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
424916834 MDExOlB1bGxSZXF1ZXN0MjY0MTMxNTU3 2850 More informative error when writing attrs to netCDF fmaussion 10050469 closed 0     8 2019-03-25T13:58:30Z 2022-05-03T16:16:21Z 2022-05-03T16:16:21Z MEMBER   0 pydata/xarray/pulls/2850

Closes #3080

Some attribute names aren't valid netCDF4 keys. For example:

python import xarray as xr ds = xr.Dataset({'x': ('y', [1, 2, 3], {'CLASS': 'foo'})}) ds.to_netcdf('test.nc')

will fail with:

NetCDF: String match to name in use

This is hard to debug if you don't know which attribute is faulty. With this small change the error should be more informative:

AttributeError: The following exception occurred when attempting to set attribute (CLASS, foo): "NetCDF: String match to name in use"

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2850/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1051366862 PR_kwDOAMm_X84ubTA8 5978 Add "see also" in to_dataframe docs fmaussion 10050469 closed 0     0 2021-11-11T21:15:15Z 2021-11-13T17:35:43Z 2021-11-13T17:35:43Z MEMBER   0 pydata/xarray/pulls/5978

A very modest contribution... I miss contributing to xarray!

Believe it or not, I did not know about to_pandas until today...

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5978/reactions",
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
384004189 MDExOlB1bGxSZXF1ZXN0MjMzMzI4OTc4 2571 CF: also decode time bounds when available fmaussion 10050469 closed 0     7 2018-11-24T16:50:13Z 2018-12-19T17:19:05Z 2018-12-19T17:19:05Z MEMBER   0 pydata/xarray/pulls/2571

Not sure if this is the best way to handle it, but it seems to work

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2571/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
379410537 MDExOlB1bGxSZXF1ZXN0MjI5ODgyNDY1 2552 Attempt to reproduce HDF error fmaussion 10050469 closed 0     1 2018-11-10T10:22:47Z 2018-11-11T12:10:46Z 2018-11-11T12:10:46Z MEMBER   0 pydata/xarray/pulls/2552

This is just a test to see if I can reproduce an HDF error reported in https://github.com/pydata/xarray/issues/2551

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2552/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
372213595 MDExOlB1bGxSZXF1ZXN0MjI0NDc1ODA5 2498 Small fix in rasterio docs fmaussion 10050469 closed 0     1 2018-10-20T13:48:51Z 2018-10-22T00:01:09Z 2018-10-22T00:01:07Z MEMBER   0 pydata/xarray/pulls/2498 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2498/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
362158655 MDExOlB1bGxSZXF1ZXN0MjE2OTQ4OTgx 2425 Plotting: restore xyincrease kwarg default to True fmaussion 10050469 closed 0     2 2018-09-20T12:12:48Z 2018-09-21T17:36:23Z 2018-09-21T17:36:21Z MEMBER   0 pydata/xarray/pulls/2425

https://github.com/pydata/xarray/pull/2294 introduced the new xincrease and yincrease kwargs and a behavior documented as:

xincrease : None, True, or False, optional Should the values on the x axes be increasing from left to right? if None, use the default for the matplotlib function.

The default was None, I suggest to set it to True: it was the default before https://github.com/pydata/xarray/pull/2294 and I think it is what most users expect.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2425/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
344883324 MDExOlB1bGxSZXF1ZXN0MjA0MTcyOTI5 2317 Remove test on rasterio rc and test for 0.36 instead fmaussion 10050469 closed 0     1 2018-07-26T15:03:20Z 2018-07-30T11:05:37Z 2018-07-30T11:05:30Z MEMBER   0 pydata/xarray/pulls/2317 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2317/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
344096196 MDExOlB1bGxSZXF1ZXN0MjAzNTcyNTkw 2310 Make RTD builds faster fmaussion 10050469 closed 0     4 2018-07-24T15:45:17Z 2018-07-27T08:20:43Z 2018-07-27T08:20:43Z MEMBER   0 pydata/xarray/pulls/2310

See https://github.com/pydata/xarray/issues/2306

This makes the builds a little bit faster. But we are still very close to the 900 s limit.

I tried to pin more packages this morning but this didn't work out. I'll merge this first (the latest build worked with this config, https://readthedocs.org/projects/xray/builds/) and try again at a later stage. 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2310/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
343260451 MDExOlB1bGxSZXF1ZXN0MjAyOTY2Mjgx 2303 Rename "Recipes" to "Gallery" fmaussion 10050469 closed 0     1 2018-07-20T22:03:27Z 2018-07-23T16:33:08Z 2018-07-23T16:33:08Z MEMBER   0 pydata/xarray/pulls/2303

Simply because it makes more sense after all.

The link address won't change so it's not a biggie (http://xarray.pydata.org/en/latest/auto_gallery/index.html)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2303/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
337202341 MDExOlB1bGxSZXF1ZXN0MTk4NDY5MjMw 2260 Plotting: do not check for monotonicity with 2D coords fmaussion 10050469 closed 0     3 2018-06-30T09:47:20Z 2018-07-03T09:17:02Z 2018-07-03T09:16:27Z MEMBER   0 pydata/xarray/pulls/2260 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2260/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
323944717 MDExOlB1bGxSZXF1ZXN0MTg4Njc2Nzgw 2146 Add favicon to docs? fmaussion 10050469 closed 0     3 2018-05-17T09:35:21Z 2018-05-19T20:37:18Z 2018-05-17T16:55:31Z MEMBER   0 pydata/xarray/pulls/2146

Don't know if we want this, but it's possible to replace the default RTD icon. I'm not even sure if the cool kids are using bookmarks anymore, but I still do. This is how it looks like on my Firefox fav tab:

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2146/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
318827030 MDExOlB1bGxSZXF1ZXN0MTg0OTA4ODE2 2091 DOC: uniformize variable names in indexing.rst fmaussion 10050469 closed 0     1 2018-04-30T09:11:14Z 2018-04-30T17:17:53Z 2018-04-30T17:17:53Z MEMBER   0 pydata/xarray/pulls/2091

https://github.com/pydata/xarray/commit/6402391cf206fd04c12d44773fecd9b42ea0c246 overwrote an array which was needed later on. In general the whole page was a bit messy with the same initial array being forgotten and then re-used much later.

This PR attempts to uniformize the variable names in order to make self-consistent subsection examples. I hope I didn't brake something on the way, but it looks ok on my local build.

(also renamed a gallery example plot which wasn't showing up correctly)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2091/reactions",
    "total_count": 3,
    "+1": 3,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
308284379 MDExOlB1bGxSZXF1ZXN0MTc3MjQ5OTcx 2015 Fix an overflow bug in decode_cf_datetime fmaussion 10050469 closed 0     4 2018-03-24T17:54:36Z 2018-03-31T01:16:15Z 2018-03-31T01:16:14Z MEMBER   0 pydata/xarray/pulls/2015

Not sure yet if this is the best way to do this.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2015/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
302012794 MDExOlB1bGxSZXF1ZXN0MTcyNjc1Nzgy 1958 Update some packages on RTD fmaussion 10050469 closed 0     0 2018-03-03T16:41:00Z 2018-03-03T16:41:10Z 2018-03-03T16:41:10Z MEMBER   0 pydata/xarray/pulls/1958 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1958/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
296124381 MDExOlB1bGxSZXF1ZXN0MTY4NDE1NjA1 1902 _color_palette consistent with or without seaborn fmaussion 10050469 closed 0     1 2018-02-10T19:00:59Z 2018-02-16T21:08:32Z 2018-02-16T21:08:32Z MEMBER   0 pydata/xarray/pulls/1902

Instead of relying on seaborn per default we now use mpl and fall back to seaborn if cmap is not recognized (e.g. with cmap="husl").

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1902/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
294777470 MDExOlB1bGxSZXF1ZXN0MTY3NDI3NjA5 1890 Simplify some rasterio tests fmaussion 10050469 closed 0     0 2018-02-06T14:24:34Z 2018-02-07T08:40:34Z 2018-02-07T08:40:34Z MEMBER   0 pydata/xarray/pulls/1890

This PR restores the tests that were incorrectly removed in https://github.com/pydata/xarray/pull/1817 and adds the what's new entry I forgot in https://github.com/pydata/xarray/pull/1712

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1890/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
294779019 MDExOlB1bGxSZXF1ZXN0MTY3NDI4Nzgx 1891 Use pip install -e in contributing docs fmaussion 10050469 closed 0     0 2018-02-06T14:28:52Z 2018-02-06T19:59:04Z 2018-02-06T19:59:04Z MEMBER   0 pydata/xarray/pulls/1891

I think that pip install is the recommended way to install packages in dev mode (see also http://www.python3statement.org/practicalities/ )

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1891/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
293869118 MDExOlB1bGxSZXF1ZXN0MTY2Nzg3ODg4 1881 Fix rasterio example in docs fmaussion 10050469 closed 0     3 2018-02-02T12:13:33Z 2018-02-02T19:11:01Z 2018-02-02T19:10:54Z MEMBER   0 pydata/xarray/pulls/1881

https://github.com/pydata/xarray/pull/1796 introduced a bug in the doc gallery. This PR reverts the code to the previous greyscale example and adds a new case using imshow (the use case is different, as I tried to explain in the descriptions).

I also took care of https://github.com/pydata/xarray/issues/1789#issuecomment-356068358 : the docs should now build even when rasterio is not installed.

cc @Zac-HD , @shoyer 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1881/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
273268690 MDExOlB1bGxSZXF1ZXN0MTUyMTI1Njky 1712 Use rasterio's transform instead of homemade coordinates fmaussion 10050469 closed 0     10 2017-11-12T21:52:04Z 2018-01-26T13:51:15Z 2018-01-26T13:50:54Z MEMBER   0 pydata/xarray/pulls/1712
  • [x] Closes https://github.com/pydata/xarray/issues/1686
  • [x] Tests added / passed
  • [x] Passes git diff upstream/master **/*py | flake8 --diff
  • [ ] Fully documented, including whats-new.rst for all changes and api.rst for new API
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1712/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
285706946 MDExOlB1bGxSZXF1ZXN0MTYwOTI1NDE0 1808 Add gallery example for multiple lines plot fmaussion 10050469 closed 0     1 2018-01-03T14:45:48Z 2018-01-03T18:34:40Z 2018-01-03T18:34:40Z MEMBER   0 pydata/xarray/pulls/1808

Extends https://github.com/pydata/xarray/pull/1804 by adding some usage examples to the gallery.

Here is how the plot looks like:

cc @dcherian and @shoyer 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1808/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
158902764 MDExOlB1bGxSZXF1ZXN0NzI4NjAzMDI= 872 ENH: more control on colorbar fmaussion 10050469 closed 0     10 2016-06-07T11:44:52Z 2017-08-10T15:49:55Z 2016-06-09T15:52:46Z MEMBER   0 pydata/xarray/pulls/872

Addresses https://github.com/pydata/xarray/issues/752 and allows to pass kwargs to colorbar. For example, it is now possible to do:

``` python import numpy as np import matplotlib.pyplot as plt import xarray as xr

x, y = np.meshgrid(np.arange(12), np.arange(12)) z = xr.DataArray(np.sqrt(x2 + y2)) ds = z.to_dataset(name='z')

fig, ((ax1, ax2), (ax3, ax4)) = plt.subplots(2, 2, figsize=(12, 12)) ds.z.plot.contourf(ax=ax1) ds.z.plot.contourf(ax=ax2, cbar_kwargs={'orientation':'horizontal', 'label':'MyLabel'}) ds.z.plot.contourf(ax=ax3, cbar_ax=ax4, cbar_kwargs={'orientation':'horizontal', 'label':'Funny Cbar', 'drawedges':True}) plt.tight_layout() plt.show() ```

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/872/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
248247382 MDExOlB1bGxSZXF1ZXN0MTM0MzI2NTA4 1502 Fix rasterio builds in docs fmaussion 10050469 closed 0     1 2017-08-06T13:30:39Z 2017-08-07T09:21:39Z 2017-08-07T09:21:39Z MEMBER   0 pydata/xarray/pulls/1502
  • [ ] Closes #xxxx
  • [ ] Tests added / passed
  • [ ] Passes git diff upstream/master | flake8 --diff
  • [ ] Fully documented, including whats-new.rst for all changes and api.rst for new API

The pinned version of rasterio we use for the doc builds isn't available anymore: https://github.com/conda-forge/rasterio-feedstock/pull/36

Tests: - builds: https://readthedocs.org/projects/xray/builds/5796746/ - displays correctly: http://xarray.pydata.org/en/fix-docs/auto_gallery/plot_rasterio.html#sphx-glr-auto-gallery-plot-rasterio-py

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1502/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
234242845 MDExOlB1bGxSZXF1ZXN0MTI0NDQ0ODUx 1445 DOC: add rasterio to build environment fmaussion 10050469 closed 0     0 2017-06-07T15:08:05Z 2017-06-07T15:11:08Z 2017-06-07T15:11:07Z MEMBER   0 pydata/xarray/pulls/1445

This is needed for the doc recipe to run

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1445/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
233849342 MDExOlB1bGxSZXF1ZXN0MTI0MTU5MzM0 1443 DOC: add salem to the list of projects extending xarray fmaussion 10050469 closed 0     0 2017-06-06T10:37:50Z 2017-06-07T11:59:52Z 2017-06-07T11:57:35Z MEMBER   0 pydata/xarray/pulls/1443

Salem is likely to remain a small project but it has a more-than-one user base and might provide some inspiration for the future pangeo projects.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1443/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
206905158 MDExOlB1bGxSZXF1ZXN0MTA1NzA0NzU3 1260 Add RasterIO backend fmaussion 10050469 closed 0     38 2017-02-10T20:59:31Z 2017-06-06T16:44:43Z 2017-06-06T10:25:22Z MEMBER   0 pydata/xarray/pulls/1260

Follow-up to https://github.com/pydata/xarray/pull/1070

This is my first backend so this is going to be a bit more work than I expected, but with your help we should be able to get through this.

A long todo list: - [x] closes https://github.com/pydata/xarray/issues/790 - [x] add tests - [x] passes git diff upstream/master | flake8 --diff - [x] whatsnew entry - [x] wrap getitem for lazy data read - [x] ~~make coordinates variables lazy too~~

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1260/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
232222633 MDExOlB1bGxSZXF1ZXN0MTIzMDI3OTEw 1432 Add dask specific kwargs to DataArray.chunk() fmaussion 10050469 closed 0     0 2017-05-30T11:30:27Z 2017-05-30T18:04:02Z 2017-05-30T16:50:02Z MEMBER   0 pydata/xarray/pulls/1432
  • [ ] Closes #xxxx
  • [ ] Tests added / passed
  • [x] Passes git diff upstream/master | flake8 --diff
  • [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

this is needed for the Rasterio PR (https://github.com/pydata/xarray/pull/1260#discussion_r118824181)

There was no test for these functionalities in the xarray test suite for Dataset either, so I just added a dummy test for one of the keywords.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1432/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
230219330 MDExOlB1bGxSZXF1ZXN0MTIxNjU5OTY0 1419 Add sphinx-gallery to the docs fmaussion 10050469 closed 0     6 2017-05-21T13:00:44Z 2017-05-24T15:20:41Z 2017-05-24T15:20:41Z MEMBER   0 pydata/xarray/pulls/1419
  • [x] Closes https://github.com/pydata/xarray/issues/1061
  • [ ] Tests added / passed
  • [ ] Passes git diff upstream/master | flake8 --diff
  • [x] Fully documented, including whats-new.rst for all changes and api.rst for new API

This uses sphinx-gallery to illustrate some examples of the xarray workflow. See it rendered here

It's nicer for the gallery if there is a plot to draw in the end but it doesn't have to be a plot. Once the gallery gets more examples it's possible to sort them by topic (see the sphinx gallery documentation for an overview of the possibilities).

I could think of several other things we could add (including moving or copying @jhamman and @rabernat 's examples to the gallery), but I thought it's better to merge quickly in order to encourage the community to contribute with more examples before the next release.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1419/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
185181822 MDExOlB1bGxSZXF1ZXN0OTA4NTM1ODU= 1060 Remove obsolete NetCDF4 Error catch fmaussion 10050469 closed 0     7 2016-10-25T17:25:33Z 2017-05-22T08:58:21Z 2017-05-22T08:58:21Z MEMBER   0 pydata/xarray/pulls/1060

[edit: this original PR was replaced with another change: "remove obsolete NetCDF4 Error catch"]

This allows to give a customized NetCDF4 object to the DataStore.

I needed this for Salem's diagnostic variables: https://github.com/fmaussion/salem/blob/master/salem/sio.py#L737-L751

My workaround (a very shallow subclass of NetCDF4DataStore) is fine for me, so I won't be offended if you decide not to merge.

On a related issue, the IndexError catch which happens here is also giving me trouble, since it hides bugs in my code.

I know I'm kind of misusing the backend, but I find it very useful the way it is. I'm open to any suggestion to make it more elegant. Thanks!

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1060/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
230211583 MDExOlB1bGxSZXF1ZXN0MTIxNjU1NzE3 1417 Update weather data example fmaussion 10050469 closed 0     1 2017-05-21T10:31:06Z 2017-05-21T19:03:04Z 2017-05-21T10:33:08Z MEMBER   0 pydata/xarray/pulls/1417
  • [ ] Closes #xxxx
  • [ ] Tests added / passed
  • [ ] Passes git diff upstream/master | flake8 --diff
  • [ ] Fully documented, including whats-new.rst for all changes and api.rst for new API

For some reason a plot from the examples was rendered statically instead of live. This is now fixed, together with an update of pandas which prevents reshaping operations on indexes.

Tested here

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1417/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
227974548 MDExOlB1bGxSZXF1ZXN0MTIwMDk0MDk4 1404 Fix pandas.tslib deprecation warning fmaussion 10050469 closed 0     0 2017-05-11T12:29:13Z 2017-05-11T16:46:23Z 2017-05-11T16:46:23Z MEMBER   0 pydata/xarray/pulls/1404
  • [ ] closes #xxxx
  • [ ] tests added / passed
  • [ ] passes git diff upstream/master | flake8 --diff
  • [ ] whatsnew entry

Since the last pandas update I get:

/home/mowglie/Documents/git/xarray/xarray/core/formatting.py:16: FutureWarning: The pandas.tslib module is deprecated and will be removed in a future version. from pandas.tslib import OutOfBoundsDatetime

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1404/reactions",
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
227807405 MDExOlB1bGxSZXF1ZXN0MTE5OTc2Mjk4 1402 Adds bottleneck to the test suite fmaussion 10050469 closed 0     1 2017-05-10T20:54:04Z 2017-05-10T23:11:06Z 2017-05-10T23:10:58Z MEMBER   0 pydata/xarray/pulls/1402

I don't know if we need to install all packages to cover all the tests which rely on bottleneck, but this was the easiest. 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1402/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
222143177 MDExOlB1bGxSZXF1ZXN0MTE2MTQxODc0 1377 Remove debug print() fmaussion 10050469 closed 0     1 2017-04-17T14:29:42Z 2017-04-17T17:45:46Z 2017-04-17T17:45:44Z MEMBER   0 pydata/xarray/pulls/1377

This was introduced by https://github.com/pydata/xarray/pull/1368 (@shoyer) 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1377/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
211943192 MDExOlB1bGxSZXF1ZXN0MTA5MTQyODc1 1296 Clearer terminology for coordinate variables fmaussion 10050469 closed 0     5 2017-03-05T10:08:41Z 2017-03-15T16:28:12Z 2017-03-15T16:28:12Z MEMBER   0 pydata/xarray/pulls/1296 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1296/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
203906493 MDExOlB1bGxSZXF1ZXN0MTAzNjY2MDUy 1235 Use LooseVersion for bottleneck checks fmaussion 10050469 closed 0     0 2017-01-29T23:54:16Z 2017-01-30T00:10:40Z 2017-01-30T00:10:40Z MEMBER   0 pydata/xarray/pulls/1235 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1235/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
202966756 MDExOlB1bGxSZXF1ZXN0MTAzMDIzMTEz 1227 mpl v2 for the docs fmaussion 10050469 closed 0     0 2017-01-24T23:04:55Z 2017-01-25T00:00:55Z 2017-01-25T00:00:55Z MEMBER   0 pydata/xarray/pulls/1227

For some reason I can't reach the webpage but @shoyer seems to be able to do so: https://github.com/pydata/xarray/issues/1167#issuecomment-274966778

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1227/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
202124581 MDExOlB1bGxSZXF1ZXN0MTAyNDUxMjE3 1222 New testing module and tests refactor fmaussion 10050469 closed 0     8 2017-01-20T12:04:01Z 2017-01-24T21:18:20Z 2017-01-24T21:18:20Z MEMBER   0 pydata/xarray/pulls/1222 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1222/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
201884828 MDExOlB1bGxSZXF1ZXN0MTAyMjgzMTk1 1220 Fix docs fmaussion 10050469 closed 0     5 2017-01-19T15:04:34Z 2017-01-20T17:05:54Z 2017-01-20T08:48:45Z MEMBER   0 pydata/xarray/pulls/1220

This PR fixes a problem we had with ipython on RTD: https://github.com/ipython/ipython/issues/8733

We this simple workaround we can now use the latest ipython version. Thanks to @takluyver for his help!

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1220/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
200701826 MDExOlB1bGxSZXF1ZXN0MTAxNDk2NDUw 1206 RTD: fix facetted maps example fmaussion 10050469 closed 0     1 2017-01-13T18:48:08Z 2017-01-14T09:15:54Z 2017-01-14T09:15:54Z MEMBER   0 pydata/xarray/pulls/1206 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1206/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
196278181 MDExOlB1bGxSZXF1ZXN0OTg0NzU5NDU= 1171 Fix tests for upcoming matplotlib v2 fmaussion 10050469 closed 0     20 2016-12-18T14:20:19Z 2017-01-04T07:43:04Z 2017-01-04T07:43:04Z MEMBER   0 pydata/xarray/pulls/1171

Just to see what happens with the upcoming matplotlib version 2

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1171/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
198030158 MDExOlB1bGxSZXF1ZXN0OTk2NzE3Njk= 1191 Integer levels and vmin/vmax fmaussion 10050469 closed 0     2 2016-12-29T16:10:22Z 2017-01-03T09:50:14Z 2017-01-03T09:50:14Z MEMBER   0 pydata/xarray/pulls/1191

Follow-up to https://github.com/pydata/xarray/pull/1171#issuecomment-269556898

From the new docstring:

levels : int or list-like object, optional Split the colormap (cmap) into discrete color intervals. If an integer is provided, "nice" levels are chosen based on the data range: this can imply that the final number of levels is not exactly the expected one. Setting ``vmin`` and/or ``vmax`` with ``levels=N`` is equivalent to setting ``levels=np.linspace(vmin, vmax, N)``.

The logic overhead is quite simple, which is an argument in favor of this simple solution. It is consistent with mpl, as long as neither vmin or vmax are set.

It might change the outcome of some existing plots, though. Should I mention this in the "Breaking changes" section too?

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1191/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
196163136 MDExOlB1bGxSZXF1ZXN0OTg0MDk1ODA= 1169 DOC: small improvements to the netCDF docs fmaussion 10050469 closed 0     1 2016-12-16T21:58:17Z 2016-12-24T11:54:51Z 2016-12-24T11:54:51Z MEMBER   0 pydata/xarray/pulls/1169

Partly addresses https://github.com/pydata/xarray/issues/768#issuecomment-187226020 and https://github.com/pydata/xarray/issues/1154, mostly by making the tone less defensive. I also added that we recommend the netCDF format for IO. (I think that the format is powerful enough for being useful in other disciplines than geosciences).

I agree with @rabernat that the current page title (Serialization and IO) could be changed to something more accessible, but I don't know how. Two possibilities, both quite long (it looks a bit ugly on RTD): - "Reading and writing xarray data structures on disk" - "Reading/writing xarray data structures"

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1169/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
196134363 MDExOlB1bGxSZXF1ZXN0OTgzODgxNzE= 1168 Add figsize, size, and aspect arguments to plotting methods fmaussion 10050469 closed 0     3 2016-12-16T19:18:13Z 2016-12-18T22:43:19Z 2016-12-18T22:43:19Z MEMBER   0 pydata/xarray/pulls/1168

Extends and finishes https://github.com/pydata/xarray/pull/637

I chose to keep seaborn's convention for two reasons: - it doesn't break existing code - now that figsize is also available, I expect the size and aspect kwargs to be less used in non-facetgrid plots

Closes https://github.com/pydata/xarray/issues/897

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1168/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
194393404 MDExOlB1bGxSZXF1ZXN0OTcxNjM2MzU= 1160 Norm should be passed to facetgrid too fmaussion 10050469 closed 0     0 2016-12-08T17:15:32Z 2016-12-09T09:46:58Z 2016-12-09T09:46:58Z MEMBER   0 pydata/xarray/pulls/1160 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1160/reactions",
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
194322754 MDExOlB1bGxSZXF1ZXN0OTcxMTI0OTE= 1158 RTD: fix docs fmaussion 10050469 closed 0     0 2016-12-08T12:24:09Z 2016-12-08T12:24:27Z 2016-12-08T12:24:27Z MEMBER   0 pydata/xarray/pulls/1158 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1158/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
194113140 MDExOlB1bGxSZXF1ZXN0OTY5Njk2MjI= 1156 RTD: furter attempt to fix savefig fmaussion 10050469 closed 0     0 2016-12-07T17:26:07Z 2016-12-07T17:30:18Z 2016-12-07T17:30:18Z MEMBER   0 pydata/xarray/pulls/1156

This implements a more robust solution based on the location of conf.py...

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1156/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
194095134 MDExOlB1bGxSZXF1ZXN0OTY5NTYyOTQ= 1155 RTD: fix savefig dir creation fmaussion 10050469 closed 0     1 2016-12-07T16:21:01Z 2016-12-07T16:53:07Z 2016-12-07T16:38:03Z MEMBER   0 pydata/xarray/pulls/1155

I've had a hard time finding out what was going on with missing RTD plots since we updated the packages until I came across this: https://github.com/ipython/ipython/issues/8733

The proposed fix creates the @savefig directory if not there. It works locally but I'm not sure if it will work on RTD (they might use a custom build directory).

The other changes are minor tweaks to remove some build warnings and/or making use of the latest sphinx Makefile. I also removed a part of the conf.py which I think is not needed anymore.

@shoyer do you remember what the inspect.findsource monkeypatch was good for?

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1155/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
192685832 MDExOlB1bGxSZXF1ZXN0OTU5NzgyMjU= 1144 Update ipython on RTD fmaussion 10050469 closed 0     1 2016-11-30T21:33:55Z 2016-11-30T22:56:24Z 2016-11-30T22:45:41Z MEMBER   0 pydata/xarray/pulls/1144

Another attempt to solve a cryptic error: https://readthedocs.org/projects/xray/builds/4727274/

But this time we are getting closer! I've found a report on ipython (https://github.com/ipython/ipython/issues/8850), an I've been able to reproduce it locally and solve it by updating to ipython 5 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1144/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
192134438 MDExOlB1bGxSZXF1ZXN0OTU1ODkwODk= 1141 Try py3 again on RTD fmaussion 10050469 closed 0     0 2016-11-28T22:27:33Z 2016-11-28T23:38:59Z 2016-11-28T23:38:59Z MEMBER   0 pydata/xarray/pulls/1141

sorry about the many PRs :-(

Let's see what happens with this one

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1141/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
192083180 MDExOlB1bGxSZXF1ZXN0OTU1NTMyMDk= 1139 Unpin python on RTD fmaussion 10050469 closed 0     2 2016-11-28T18:35:51Z 2016-11-28T22:13:58Z 2016-11-28T20:00:17Z MEMBER   0 pydata/xarray/pulls/1139

Pinning python to 3.5 introduced new problems.

Despite the default python being 3 here and also in our RTD preferences (dashboard->advanced), RTD seems to still use py2 per default...

Anyway, going back to py2 is the easiest for now, I might come back to this when I have more time.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1139/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
191822204 MDExOlB1bGxSZXF1ZXN0OTUzODA0NDE= 1134 Further attempt to get netCDF4 working on RTD fmaussion 10050469 closed 0     4 2016-11-26T19:25:45Z 2016-11-28T11:11:24Z 2016-11-26T21:48:40Z MEMBER   0 pydata/xarray/pulls/1134

The idea came from here: https://github.com/Unidata/netcdf4-python/issues/574#issuecomment-235435628

I could test it on another repo, so I'm confident that this should work here too. Obviously, this is a temporary solution.

hopefully fixes https://github.com/pydata/xarray/issues/1106

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1134/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
191856659 MDExOlB1bGxSZXF1ZXN0OTU0MDAwNTY= 1137 Pin package versions on RTD fmaussion 10050469 closed 0     1 2016-11-27T11:38:11Z 2016-11-28T03:45:12Z 2016-11-28T03:45:11Z MEMBER   0 pydata/xarray/pulls/1137

Now that we seem to have everything working on RTD, re-pin package versions

(follow-up to https://github.com/pydata/xarray/pull/1101)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1137/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
191830253 MDExOlB1bGxSZXF1ZXN0OTUzODUyMDI= 1136 WIP: remove some refs to xray fmaussion 10050469 closed 0     1 2016-11-26T22:30:27Z 2016-11-26T23:01:53Z 2016-11-26T23:00:50Z MEMBER   0 pydata/xarray/pulls/1136

There are some uses of xray that needed to be fixed.

WIP: merge after you decide what to do with https://github.com/pydata/xarray/issues/1135

Also, I'd be in favor of removing the two notebooks in https://github.com/pydata/xarray/tree/master/examples since they are also available in the docs (http://xarray.pydata.org/en/latest/examples.html). The example on RTD have the disadvantage that it's less easy to do copy/paste with them, but notebooks are more error prone since they need to be maintained externally. 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1136/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188862291 MDExOlB1bGxSZXF1ZXN0OTM0MDY2MzM= 1112 Bug in DecodedCFDatetimeArray fmaussion 10050469 closed 0     0 2016-11-11T22:11:22Z 2016-11-13T18:04:00Z 2016-11-13T18:04:00Z MEMBER   0 pydata/xarray/pulls/1112 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1112/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188818401 MDExOlB1bGxSZXF1ZXN0OTMzNzQyNjA= 1110 Add default channel to conda on RTD fmaussion 10050469 closed 0     3 2016-11-11T18:13:25Z 2016-11-11T18:36:24Z 2016-11-11T18:36:23Z MEMBER   0 pydata/xarray/pulls/1110

As discussed here: https://github.com/pydata/xarray/issues/1106

I also added a section somewhere to print the packages versions on RTD. It will be useful to pin the versions again once it works, and is useful enough to stay on the docs I think. I just didn't really knew where to put it, other ideas welcome!

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1110/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188810255 MDExOlB1bGxSZXF1ZXN0OTMzNjgyNTU= 1108 py2 compat header fmaussion 10050469 closed 0     3 2016-11-11T17:29:05Z 2016-11-11T18:01:11Z 2016-11-11T18:01:06Z MEMBER   0 pydata/xarray/pulls/1108

As discussed in https://github.com/pydata/xarray/pull/1079

I just left out a couple of nearly empty __init__s

I also removed a couple of unused imports

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1108/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
187208913 MDExOlB1bGxSZXF1ZXN0OTIyNTM4Nzc= 1079 New infer_intervals keyword for pcolormesh fmaussion 10050469 closed 0     23 2016-11-03T22:35:29Z 2016-11-10T22:55:03Z 2016-11-10T22:55:03Z MEMBER   0 pydata/xarray/pulls/1079

Addresses https://github.com/pydata/xarray/issues/781

@jhamman what do you think? I'm not sure if infer_interval_breaks is the best name for the keyword but I didn't come up with anything better than that. (maybe infer_coord_intervals?)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1079/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188546407 MDExOlB1bGxSZXF1ZXN0OTMxNzc4NzI= 1105 Another attempt to get netCDF4 working on RTD + py3 fmaussion 10050469 closed 0     1 2016-11-10T15:59:27Z 2016-11-10T16:50:18Z 2016-11-10T16:49:42Z MEMBER   0 pydata/xarray/pulls/1105

Sorry @shoyer you might get a few of these debugging PRs today again.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1105/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188350265 MDExOlB1bGxSZXF1ZXN0OTMwMzcyNzA= 1101 unpin package versions for doc build fmaussion 10050469 closed 0     3 2016-11-09T20:55:47Z 2016-11-10T02:44:28Z 2016-11-09T21:07:01Z MEMBER   0 pydata/xarray/pulls/1101 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1101/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188125831 MDExOlB1bGxSZXF1ZXN0OTI4NzU1MjY= 1098 Docs tweaks fmaussion 10050469 closed 0     6 2016-11-08T22:50:34Z 2016-11-09T20:52:19Z 2016-11-09T17:44:49Z MEMBER   0 pydata/xarray/pulls/1098

The multidimensional coords example is now built live. I tested this locally and tried to keep the look of the figures as close as possible to the original ones.

CC @rabernat @shoyer

(I also added a link to salem in the list of packages extending xarray - shameless self-promotion)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1098/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
188340763 MDExOlB1bGxSZXF1ZXN0OTMwMzAzNDI= 1099 Add cartopy and netcdf4 to the doc build fmaussion 10050469 closed 0     0 2016-11-09T20:10:25Z 2016-11-09T20:41:04Z 2016-11-09T20:41:04Z MEMBER   0 pydata/xarray/pulls/1099 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1099/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
183218516 MDExOlB1bGxSZXF1ZXN0ODk0OTgzMjU= 1049 Preserve variables order in Dataset.concat() fmaussion 10050469 closed 0     5 2016-10-15T16:30:32Z 2016-11-02T01:35:53Z 2016-11-02T01:35:53Z MEMBER   0 pydata/xarray/pulls/1049

Alternative (and much easier) implementation to https://github.com/pydata/xarray/pull/1048

The problem with the coordinate variables remains the same

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1049/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
183199483 MDExOlB1bGxSZXF1ZXN0ODk0ODc2MDc= 1048 WIP: preserve variables order in Dataset.concat() fmaussion 10050469 closed 0     1 2016-10-15T09:27:53Z 2016-10-15T22:50:16Z 2016-10-15T22:50:16Z MEMBER   0 pydata/xarray/pulls/1048

addresses https://github.com/pydata/xarray/issues/1042

The problem is that sets aren't ordered. I tried something based on an OrderedSet recipe, but I'm not sure that this is the most elegant way to go.

A remaining issue is that the coord variable is still not in place (['t', 'a', 'b', 'c'] becomes ['a', 'b', 'c', 't']), but I'm not sure it's possible to do it better (it happens here).

Any idea welcome

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1048/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
178119989 MDExOlB1bGxSZXF1ZXN0ODYwMTc4MjY= 1011 .where() and .fillna() should preserve attributes fmaussion 10050469 closed 0     4 2016-09-20T17:11:21Z 2016-09-21T17:26:38Z 2016-09-21T17:26:32Z MEMBER   0 pydata/xarray/pulls/1011

Fixes https://github.com/pydata/xarray/issues/1009

There is probably a more elegant way to do it, but the internals of xarray still confuse me. I'm happy to learn!

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1011/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
178121292 MDExOlB1bGxSZXF1ZXN0ODYwMTg3NDg= 1012 mini improvement of the install doc fmaussion 10050469 closed 0     1 2016-09-20T17:17:14Z 2016-09-20T17:18:42Z 2016-09-20T17:18:39Z MEMBER   0 pydata/xarray/pulls/1012

cyordereddict is now obsolete

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1012/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
158412366 MDExOlB1bGxSZXF1ZXN0NzI1MzkzMDU= 868 BUG: contour plot levels fmaussion 10050469 closed 0     2 2016-06-03T17:10:09Z 2016-07-20T17:16:28Z 2016-07-20T17:16:27Z MEMBER   0 pydata/xarray/pulls/868 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/868/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
158551074 MDExOlB1bGxSZXF1ZXN0NzI2Mjc1NTE= 870 DOC: mention conda-forge fmaussion 10050469 closed 0     2 2016-06-05T11:59:45Z 2016-06-06T20:13:54Z 2016-06-06T20:13:49Z MEMBER   0 pydata/xarray/pulls/870

Follow-up on https://github.com/pydata/xarray/issues/869#issuecomment-223776182

I am no native speaker, so feel free to change everything.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/870/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
140384427 MDExOlB1bGxSZXF1ZXN0NjI2NDE0OTI= 794 fix: transpose non-linked coordinates in 2d plots fmaussion 10050469 closed 0     5 2016-03-12T13:32:59Z 2016-03-13T20:06:02Z 2016-03-13T20:06:02Z MEMBER   0 pydata/xarray/pulls/794 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/794/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
137396688 MDExOlB1bGxSZXF1ZXN0NjExNTYzNDk= 778 fix value error in data-structures doc fmaussion 10050469 closed 0     1 2016-02-29T22:21:58Z 2016-03-01T00:50:27Z 2016-03-01T00:50:27Z MEMBER   0 pydata/xarray/pulls/778

This should fix the error in the docs here: http://xarray.pydata.org/en/latest/data-structures.html

It is a detail, but it occurred to me that the examples of this section don't use the (space, time) dimension order (e.g. dims=['time', 'space']). Most of the climate datasets I know of are using the numpy (lat, lon, time) dim order. Is it intentional?

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/778/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
116349205 MDExOlB1bGxSZXF1ZXN0NTAzOTExNTk= 655 More control on cmap params fmaussion 10050469 closed 0     21 2015-11-11T14:51:10Z 2015-11-15T22:20:04Z 2015-11-15T22:19:05Z MEMBER   0 pydata/xarray/pulls/655

Motivation: https://github.com/xray/xray/issues/642

User's vmin, vmax are no longer overwritten. You can prevent divergent cmaps and keep automatic vmin, vmax bei setting center=False. This is quite intuitive I think (first thing I tried when I wanted to avoid a divergent colormap).

See the new tests for examples of use. 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/655/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options: 

CREATE TABLE [issues] (
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [number] INTEGER,
   [title] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [state] TEXT,
   [locked] INTEGER,
   [assignee] INTEGER REFERENCES [users]([id]),
   [milestone] INTEGER REFERENCES [milestones]([id]),
   [comments] INTEGER,
   [created_at] TEXT,
   [updated_at] TEXT,
   [closed_at] TEXT,
   [author_association] TEXT,
   [active_lock_reason] TEXT,
   [draft] INTEGER,
   [pull_request] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [state_reason] TEXT,
   [repo] INTEGER REFERENCES [repos]([id]),
   [type] TEXT
);
CREATE INDEX [idx_issues_repo]
    ON [issues] ([repo]);
CREATE INDEX [idx_issues_milestone]
    ON [issues] ([milestone]);
CREATE INDEX [idx_issues_assignee]
    ON [issues] ([assignee]);
CREATE INDEX [idx_issues_user]
    ON [issues] ([user]);