html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/5710#issuecomment-900384483,https://api.github.com/repos/pydata/xarray/issues/5710,900384483,IC_kwDOAMm_X841qsbj,13205162,2021-08-17T15:11:23Z,2021-08-17T15:11:23Z,CONTRIBUTOR,"On my end I think this idea is fine! Although I don't really understand why matplotlib would be needed for that test in the first place. 
Sorry to have inadvertently introduced a failing test.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,972720354
https://github.com/pydata/xarray/pull/5682#issuecomment-899947392,https://api.github.com/repos/pydata/xarray/issues/5682,899947392,IC_kwDOAMm_X841pBuA,13205162,2021-08-17T02:33:45Z,2021-08-17T02:33:45Z,CONTRIBUTOR,"> That's unrelated, see #5654

Ah, I see. Thanks for merging!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963239311
https://github.com/pydata/xarray/pull/5682#issuecomment-899943472,https://api.github.com/repos/pydata/xarray/issues/5682,899943472,IC_kwDOAMm_X841pAww,13205162,2021-08-17T02:22:01Z,2021-08-17T02:22:01Z,CONTRIBUTOR,"> Looks great. I'll merge when tests pass.
> 
> Thanks @tomchor. I see this is your first PR here. Welcome to xarray!

Thanks!

I think you spoke too soon though haha. One of the tests is failing and I don't really know why! I never touched the `test_chunk()` function...:

```
=========================== short test summary info ============================
FAILED xarray/tests/test_sparse.py::test_chunk - TypeError: operand type(s) a...
= 1 failed, 13681 passed, 1110 skipped, 225 xfailed, 67 xpassed, 212 warnings in 445.02s (0:07:25) =
**PNC:/usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/PseudoNetCDF/pncwarn.py:24:UserWarning:
  pyproj could not be found, so IO/API coordinates cannot be converted to lat/lon; to fix, install pyproj or basemap (e.g., `pip install pyproj)`
Error: Process completed with exit code 1.
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963239311
https://github.com/pydata/xarray/pull/5682#issuecomment-899085600,https://api.github.com/repos/pydata/xarray/issues/5682,899085600,IC_kwDOAMm_X841lvUg,13205162,2021-08-15T17:32:23Z,2021-08-15T17:32:23Z,CONTRIBUTOR,"> @tomchor thanks. Yes all you need to do is define a test function that checks something and pytest will run it.
> 
> you can run it following the instructions here: https://xarray.pydata.org/en/stable/contributing.html#running-the-test-suite
> 
> (something like this)
> 
> ```
> pytest xarray/tests/test_utils.py -k test_latex_name_isnt_split
> ```

Thanks! This makes it much easier to test everything locally. I test the new test locally and it should pass now.
I also saw that `test_chunk()` isn't passing and I don't know why. Also I can't run that test locally for some reason.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963239311
https://github.com/pydata/xarray/pull/5682#issuecomment-897160926,https://api.github.com/repos/pydata/xarray/issues/5682,897160926,IC_kwDOAMm_X841eZbe,13205162,2021-08-11T21:17:26Z,2021-08-11T21:17:26Z,CONTRIBUTOR,"I defined a function that tests that a long latex sequence stays intact after going through `label_from_attrs()`, but I'm not sure if that's the same as _adding the test_. It seems from the Pytest docs that you don't need to call the function explicitly; just define it. In which case that do the trick.

@dcherian could you please confirm? 

Also the tests weren't passing before I added this last commit, and I have no idea why.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963239311
https://github.com/pydata/xarray/pull/5682#issuecomment-894882015,https://api.github.com/repos/pydata/xarray/issues/5682,894882015,IC_kwDOAMm_X841VtDf,13205162,2021-08-09T00:27:12Z,2021-08-09T00:27:12Z,CONTRIBUTOR,"> I wonder if it will look best if we avoid splitting inside the latex code?
> `name`, `extra` and `units` can have latex as well so I think we need to handle the entire `name+extra+units` string somehow.
> 
> I was playing around a little with regex but I didn't get that far:
> 
> ```python
> import textwrap
> import re
> 
> name = r""$Ra_s = \mathrm{mean}(\epsilon_k) / \mu M^2_\infty$""
> extra = ""Longwinded comments about something confusing. ""
> units = r""$[N / m]$""
> label_raw = name + extra + units
> label_notex = re.sub(""\$(.+?)\$"", ""X "", label_raw, flags=re.DOTALL)
> label_notex_wrap = textwrap.wrap(label_notex, 30)
> label_tex_wrap = ... # regex to add back the latex strings.  
> ```
> 
> There might very well be a simpler way of doing this.

I couldn't figure out any way to avoid splitting LaTeX code. I suspect that the way to do it might be kind of complex (but I'm happy if someone can prove me wrong). We can also increase the number of columns for the latex option even more to maintain simplicity...","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963239311
https://github.com/pydata/xarray/issues/5681#issuecomment-894561619,https://api.github.com/repos/pydata/xarray/issues/5681,894561619,IC_kwDOAMm_X841Ue1T,13205162,2021-08-06T23:14:34Z,2021-08-06T23:14:34Z,CONTRIBUTOR,"I'm not entirely sure why that would make the LaTeX renderer fail. But if that's the case and skipping it is an option, I'd test that both the first and last characters are `$` before skipping.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963089887
https://github.com/pydata/xarray/issues/5681#issuecomment-894555968,https://api.github.com/repos/pydata/xarray/issues/5681,894555968,IC_kwDOAMm_X841UddA,13205162,2021-08-06T22:55:28Z,2021-08-06T22:55:57Z,CONTRIBUTOR,"Note that for simple latex expressions xarray appears to work fine. For example `name = r""$\mathrm{mean}(\epsilon_k)$""` works in both figures in the example above.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,963089887
https://github.com/pydata/xarray/issues/4997#issuecomment-792477146,https://api.github.com/repos/pydata/xarray/issues/4997,792477146,MDEyOklzc3VlQ29tbWVudDc5MjQ3NzE0Ng==,13205162,2021-03-08T05:33:45Z,2021-03-08T05:33:45Z,CONTRIBUTOR,"This definitely sounds okay by me. I'm trying to figure out on my own how to implement this, but these plotting functions are quite complicated with the decorators and the inferences and all! I haven't been able to make much sense of it unfortunately.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,822404281
https://github.com/pydata/xarray/issues/4212#issuecomment-775284567,https://api.github.com/repos/pydata/xarray/issues/4212,775284567,MDEyOklzc3VlQ29tbWVudDc3NTI4NDU2Nw==,13205162,2021-02-08T16:50:38Z,2021-02-08T16:50:38Z,CONTRIBUTOR,"@jacobtomlinson Really glad someone's working on this! I'd be glad to help if I can (although I've never contributed to xarray and I don't know much about GPUs). I have some questions though. 

Do you have a specific purpose in mind for this? I ask because most other discussions I see related to this really just wanna do ML. However, there's a large user base (myself included) that would benefit immensely from just doing regular (non-machine-learning) operations  with a GPU backend.

Also, what's the status on the development? I see no comments after July 2020 and I'm hoping I can help get this back on track if needed!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,654135405
https://github.com/pydata/xarray/issues/2911#issuecomment-489873332,https://api.github.com/repos/pydata/xarray/issues/2911,489873332,MDEyOklzc3VlQ29tbWVudDQ4OTg3MzMzMg==,13205162,2019-05-07T01:41:15Z,2019-05-07T01:41:15Z,CONTRIBUTOR,"Wrote a simple function [here](https://github.com/tomchor/lespy/blob/master/io.py#L215:L273).
I'm not sure what the best to link this to the docs is, since I can't guarantee that this function will change in the future. Maybe link the commit? Or maybe just straight up pasting that into the docs with a small example script using it?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,435532136
https://github.com/pydata/xarray/issues/2911#issuecomment-485473367,https://api.github.com/repos/pydata/xarray/issues/2911,485473367,MDEyOklzc3VlQ29tbWVudDQ4NTQ3MzM2Nw==,13205162,2019-04-22T16:49:46Z,2019-04-22T16:49:57Z,CONTRIBUTOR,"@dcherian That's a good idea. At the moment my implementation is very problem-specific so it's not good to upload that. Generalizing it is very straightforward though (albeit a bit of work), but I'll try to do that soon. I wonder what the best place for that in the docs would be.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,435532136
https://github.com/pydata/xarray/issues/66#issuecomment-485287247,https://api.github.com/repos/pydata/xarray/issues/66,485287247,MDEyOklzc3VlQ29tbWVudDQ4NTI4NzI0Nw==,13205162,2019-04-21T22:37:48Z,2019-04-21T22:37:48Z,CONTRIBUTOR,"Have there been any developments for HDF5 support?
I've been trying to read HDF5 data from [Dedalus](http://dedalus-project.org/) but I've been having a hard time.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,29453809
https://github.com/pydata/xarray/issues/1332#issuecomment-418154705,https://api.github.com/repos/pydata/xarray/issues/1332,418154705,MDEyOklzc3VlQ29tbWVudDQxODE1NDcwNQ==,13205162,2018-09-03T16:14:46Z,2018-09-03T16:14:46Z,CONTRIBUTOR,"I'm not sure where we stand on this issue, but I think since `numpy.gradient` already exists, it makes more sense to wrap that function for the sake of simplicity instead of making `xr.diff` differ from the original premise of `np.diff`.

On the same topic, it bothers me that `xr.diff` only accepts `""upper""` and `""lower""` arguments for the label. The most obvious (and useful) value in my opinion would be `""middle""`, which would correspond to a centered finite differences. Is there any special reason why that option isn't there?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217385961
https://github.com/pydata/xarray/issues/2286#issuecomment-405054591,https://api.github.com/repos/pydata/xarray/issues/2286,405054591,MDEyOklzc3VlQ29tbWVudDQwNTA1NDU5MQ==,13205162,2018-07-14T22:44:54Z,2018-07-14T22:44:54Z,CONTRIBUTOR,Just realized that you can do that with successive iterations of `combine_first`. I had misread the docs previously. This can be considered a closed issue.,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,341272087
https://github.com/pydata/xarray/issues/2035#issuecomment-378123085,https://api.github.com/repos/pydata/xarray/issues/2035,378123085,MDEyOklzc3VlQ29tbWVudDM3ODEyMzA4NQ==,13205162,2018-04-03T04:15:29Z,2018-04-03T04:15:29Z,CONTRIBUTOR,@maxim-lian But my last point remains: should this really return a zero-valued matrix? I feel like this is counter-intuitive,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,310670696
https://github.com/pydata/xarray/issues/2035#issuecomment-378114226,https://api.github.com/repos/pydata/xarray/issues/2035,378114226,MDEyOklzc3VlQ29tbWVudDM3ODExNDIyNg==,13205162,2018-04-03T03:08:07Z,2018-04-03T03:08:07Z,CONTRIBUTOR,"I just realized this is probably the expected behavior (although counter-intuitive for me) since the coordinates of the points I'm operating on are different. I still don't understand why the returned array has zero values though, instead of nans.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,310670696
https://github.com/pydata/xarray/issues/2030#issuecomment-377571072,https://api.github.com/repos/pydata/xarray/issues/2030,377571072,MDEyOklzc3VlQ29tbWVudDM3NzU3MTA3Mg==,13205162,2018-03-30T17:02:40Z,2018-03-30T17:03:09Z,CONTRIBUTOR,"@philippjfr Even with the alteration I still get a `HoloMap type not found` (using HV 1.9.5) error. I can still see the animation, though. And switching to 'Agg' did allow me to save the animation in mp4, which is good. All workarounds aside, this works pretty well. Thanks a lot to all.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,309965118
https://github.com/pydata/xarray/issues/2030#issuecomment-377565812,https://api.github.com/repos/pydata/xarray/issues/2030,377565812,MDEyOklzc3VlQ29tbWVudDM3NzU2NTgxMg==,13205162,2018-03-30T16:40:02Z,2018-03-30T16:40:02Z,CONTRIBUTOR,"@philippjfr Thanks for the comment. I hand't found anything like that. For me, `%%output` isn't a recognized magic command, and the `renderer.save` command fails for me with a tkinter-related error (can't invoke ""wm"" command), so I really couldn't see anything yet. But I believe I'll be able to solve that eventually.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,309965118
https://github.com/pydata/xarray/issues/2030#issuecomment-377543021,https://api.github.com/repos/pydata/xarray/issues/2030,377543021,MDEyOklzc3VlQ29tbWVudDM3NzU0MzAyMQ==,13205162,2018-03-30T15:00:44Z,2018-03-30T15:00:44Z,CONTRIBUTOR,"I didn't know holoviews existed, thanks for pointing that out to me. The fact that it can't create an animation file (as far as I could tell so far) does mean I can't use it, though. So I'll still try to pursue the matplotlib animation option, although maybe for most users holoviews+xarray might already be enough (I don't know).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,309965118
https://github.com/pydata/xarray/issues/2030#issuecomment-377439646,https://api.github.com/repos/pydata/xarray/issues/2030,377439646,MDEyOklzc3VlQ29tbWVudDM3NzQzOTY0Ng==,13205162,2018-03-30T04:02:24Z,2018-03-30T04:16:17Z,CONTRIBUTOR,"Unfortunately I don't have any example with DataArray right now. Since I never could take advantage of DataArray's plotting capabilities for animations, I always did animations using pure Numpy. However, I'm talking about the standard matplotlib animations. Here's an example taken from [here](https://matplotlib.org/examples/animation/basic_example_writer.html):

```import numpy as np
import matplotlib
matplotlib.use(""Agg"")
import matplotlib.pyplot as plt
import matplotlib.animation as animation


def update_line(num, data, line):
    line.set_data(data[..., :num])
    return line,

# Set up formatting for the movie files
Writer = animation.writers['ffmpeg']
writer = Writer(fps=15, metadata=dict(artist='Me'), bitrate=1800)


fig1 = plt.figure()

data = np.random.rand(2, 25)
l, = plt.plot([], [], 'r-')
plt.xlim(0, 1)
plt.ylim(0, 1)
plt.xlabel('x')
plt.title('test')
line_ani = animation.FuncAnimation(fig1, update_line, 25, fargs=(data, l),
                                   interval=50, blit=True)
line_ani.save('lines.mp4', writer=writer)
```

If you have some directions on a smart way to bring xarray into the picture, maybe I can try to come up with an example that might evolve into a contribution.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,309965118