issue_comments

Table actions

45 rows where issue = 526273373 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: reactions, created_at (date), updated_at (date)

user 4

issue 1

  • merge scipy19 docs · 45 ✖

author_association 1

id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
557343738 https://github.com/pydata/xarray/pull/3557#issuecomment-557343738 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzM0MzczOA== dcherian 2448579 2019-11-22T01:14:56Z 2019-11-22T01:14:56Z MEMBER

Great work as usual @keewis !

 
{
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 1,
    "eyes": 0
}
   merge scipy19 docs 526273373
557325251 https://github.com/pydata/xarray/pull/3557#issuecomment-557325251 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMyNTI1MQ== keewis 14808389 2019-11-21T23:54:15Z 2019-11-21T23:54:15Z MEMBER

I think we can merge after making sure RTD still builds?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557323177 https://github.com/pydata/xarray/pull/3557#issuecomment-557323177 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMyMzE3Nw== keewis 14808389 2019-11-21T23:45:15Z 2019-11-21T23:45:15Z MEMBER

done

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557320384 https://github.com/pydata/xarray/pull/3557#issuecomment-557320384 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMyMDM4NA== dcherian 2448579 2019-11-21T23:33:47Z 2019-11-21T23:33:47Z MEMBER

:+1:

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557316984 https://github.com/pydata/xarray/pull/3557#issuecomment-557316984 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxNjk4NA== keewis 14808389 2019-11-21T23:21:55Z 2019-11-21T23:21:55Z MEMBER

how about - Switch doc examples to use nbsphinx and replace sphinx_gallery with notebook. (:pull:`3105`, :pull:`3106`, :pull:`3121`) By `Ryan Abernathey <https://github.com/rabernat>` - Added example notebook demonstrating the visualization of ERA5 GRIB data. (:pull:`3199`). By `Zach Bruick <https://github.com/zbruick>` and `Stephan Siemen <https://github.com/StephanSiemen>`

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557314845 https://github.com/pydata/xarray/pull/3557#issuecomment-557314845 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxNDg0NQ== dcherian 2448579 2019-11-21T23:14:27Z 2019-11-21T23:14:27Z MEMBER

Yes let's add appropriate entries for all those. 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557314575 https://github.com/pydata/xarray/pull/3557#issuecomment-557314575 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxNDU3NQ== keewis 14808389 2019-11-21T23:13:18Z 2019-11-21T23:13:27Z MEMBER

while we have an entry for #3116 the one for #3199 is missing (also, @rabernat, do you want an entry for switching to nbsphinx?)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557314539 https://github.com/pydata/xarray/pull/3557#issuecomment-557314539 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxNDUzOQ== dcherian 2448579 2019-11-21T23:13:09Z 2019-11-21T23:13:09Z MEMBER

Documentation

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557313484 https://github.com/pydata/xarray/pull/3557#issuecomment-557313484 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxMzQ4NA== keewis 14808389 2019-11-21T23:09:08Z 2019-11-21T23:09:08Z MEMBER

would that be documentation or enhancements?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557312589 https://github.com/pydata/xarray/pull/3557#issuecomment-557312589 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxMjU4OQ== dcherian 2448579 2019-11-21T23:05:31Z 2019-11-21T23:05:31Z MEMBER

Yes! there seems to be one new example entry

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557311995 https://github.com/pydata/xarray/pull/3557#issuecomment-557311995 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxMTk5NQ== keewis 14808389 2019-11-21T23:03:35Z 2019-11-21T23:03:35Z MEMBER

ah, maybe we should move the whats-new.rst entries?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557310921 https://github.com/pydata/xarray/pull/3557#issuecomment-557310921 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMxMDkyMQ== dcherian 2448579 2019-11-21T23:00:10Z 2019-11-21T23:00:10Z MEMBER

OK I jsut got too excited.

 
{
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 1,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557309786 https://github.com/pydata/xarray/pull/3557#issuecomment-557309786 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwOTc4Ng== max-sixty 5635139 2019-11-21T22:56:31Z 2019-11-21T22:56:31Z MEMBER

Should we change the base of this PR to master? Then we can choose when we push to stable? (But let's not get stuck on this, fine to just go ahead if we can't resolve)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557309528 https://github.com/pydata/xarray/pull/3557#issuecomment-557309528 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwOTUyOA== keewis 14808389 2019-11-21T22:55:39Z 2019-11-21T22:55:39Z MEMBER

@dcherian, I'd merge into master and then cherry-pick into stable?

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557309056 https://github.com/pydata/xarray/pull/3557#issuecomment-557309056 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwOTA1Ng== keewis 14808389 2019-11-21T22:54:16Z 2019-11-21T22:54:16Z MEMBER

yeah, should we merge to stable we should probably rebase first

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557308675 https://github.com/pydata/xarray/pull/3557#issuecomment-557308675 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwODY3NQ== max-sixty 5635139 2019-11-21T22:53:09Z 2019-11-21T22:53:09Z MEMBER

Feel free to ignore since it's not a big deal relative to the benefits, but looks like this adds some recent changes on master to stable (e.g. https://github.com/pydata/xarray/pull/3557/files#diff-99289912975ea19afc02f51c240ca451R18)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557307923 https://github.com/pydata/xarray/pull/3557#issuecomment-557307923 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwNzkyMw== keewis 14808389 2019-11-21T22:50:33Z 2019-11-21T22:50:33Z MEMBER

one thing that might be done in a future PR is enabling the HTML reprs?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557305287 https://github.com/pydata/xarray/pull/3557#issuecomment-557305287 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzMwNTI4Nw== keewis 14808389 2019-11-21T22:41:21Z 2019-11-21T22:41:21Z MEMBER

I would think so, but I think we can fix anything that pops up later in new PRs.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557296512 https://github.com/pydata/xarray/pull/3557#issuecomment-557296512 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI5NjUxMg== dcherian 2448579 2019-11-21T22:13:30Z 2019-11-21T22:13:30Z MEMBER

Parsing rasterio geocoordinates? :man_shrugging:

Is that the last thing to fix?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557295586 https://github.com/pydata/xarray/pull/3557#issuecomment-557295586 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI5NTU4Ng== dcherian 2448579 2019-11-21T22:10:48Z 2019-11-21T22:10:48Z MEMBER

I just switched it so we can tweet about binderable examples ;) 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557293832 https://github.com/pydata/xarray/pull/3557#issuecomment-557293832 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI5MzgzMg== keewis 14808389 2019-11-21T22:05:36Z 2019-11-21T22:09:16Z MEMBER

okay. How about using Parsing the geocoordinates of rasterio or Parsing the rasterio geocoordinates?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557293969 https://github.com/pydata/xarray/pull/3557#issuecomment-557293969 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI5Mzk2OQ== keewis 14808389 2019-11-21T22:06:04Z 2019-11-21T22:06:04Z MEMBER

oh, and is there a reason why we merge into stable instead of master?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557290590 https://github.com/pydata/xarray/pull/3557#issuecomment-557290590 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI5MDU5MA== dcherian 2448579 2019-11-21T21:57:03Z 2019-11-21T21:57:03Z MEMBER

Let's just rename this heading (/examples/visualization_gallery.ipynb#parsing-rasterio%u2019s-geocoordinates) and merge?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557285846 https://github.com/pydata/xarray/pull/3557#issuecomment-557285846 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI4NTg0Ng== dcherian 2448579 2019-11-21T21:44:20Z 2019-11-21T21:44:20Z MEMBER

:tada: :tada: :tada: :tada:

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557285583 https://github.com/pydata/xarray/pull/3557#issuecomment-557285583 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI4NTU4Mw== dcherian 2448579 2019-11-21T21:43:46Z 2019-11-21T21:43:46Z MEMBER 
{
    "total_count": 2,
    "+1": 0,
    "-1": 0,
    "laugh": 1,
    "hooray": 1,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557285500 https://github.com/pydata/xarray/pull/3557#issuecomment-557285500 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI4NTUwMA== keewis 14808389 2019-11-21T21:43:33Z 2019-11-21T21:43:33Z MEMBER

the shadowing was avoided by using pip for installing cfgrib (otherwise it would have pulled a packaged version of xarray). Anyways, the build seems to have been successfully completed

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557284945 https://github.com/pydata/xarray/pull/3557#issuecomment-557284945 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI4NDk0NQ== dcherian 2448579 2019-11-21T21:42:10Z 2019-11-21T21:42:10Z MEMBER

My recollection is that we had some kind of version shadowing problem so maybe now is the time to replace versioneer: https://github.com/pydata/xarray/issues/2853

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557279469 https://github.com/pydata/xarray/pull/3557#issuecomment-557279469 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI3OTQ2OQ== dcherian 2448579 2019-11-21T21:27:39Z 2019-11-21T21:27:39Z MEMBER

OK it's going here: https://readthedocs.org/projects/xray/builds/10000789/

I renamed the existing scipy19-docs branch to scipy19-docs-backup and force pushed your branch to scipy19-docs

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557278022 https://github.com/pydata/xarray/pull/3557#issuecomment-557278022 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI3ODAyMg== keewis 14808389 2019-11-21T21:23:39Z 2019-11-21T21:23:39Z MEMBER

also, there is one issue with the labels: this /examples/visualization_gallery.ipynb#parsing-rasterio%u2019s-geocoordinates does not work (because of the unicode character I think).

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557276254 https://github.com/pydata/xarray/pull/3557#issuecomment-557276254 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzI3NjI1NA== keewis 14808389 2019-11-21T21:18:53Z 2019-11-21T21:21:05Z MEMBER

I added the changes from #3464 to the notebook. @rabernat, do you have the time to check whether the cherry-picks were applied correctly?

@dcherian, do you think you can try out this branch on the official RTD setup that (apparently) has more resources? I don't think I can make my setup work without a lot of effort.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557208666 https://github.com/pydata/xarray/pull/3557#issuecomment-557208666 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzIwODY2Ng== keewis 14808389 2019-11-21T18:16:28Z 2019-11-21T18:16:28Z MEMBER

oh, and we have some warnings regarding missing document labels. Is there a way to reference labels in notebooks?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557205607 https://github.com/pydata/xarray/pull/3557#issuecomment-557205607 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzIwNTYwNw== keewis 14808389 2019-11-21T18:08:24Z 2019-11-21T18:08:24Z MEMBER

now that CI passes we only need to get the readthedocs build to work (and maybe re-apply #3464?).

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557196694 https://github.com/pydata/xarray/pull/3557#issuecomment-557196694 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE5NjY5NA== dcherian 2448579 2019-11-21T17:45:26Z 2019-11-21T17:45:26Z MEMBER 

{
    "total_count": 3,
    "+1": 1,
    "-1": 0,
    "laugh": 2,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557181868 https://github.com/pydata/xarray/pull/3557#issuecomment-557181868 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE4MTg2OA== keewis 14808389 2019-11-21T17:09:07Z 2019-11-21T17:09:07Z MEMBER

iris from github would work, but cartopy has no wheels, so installing using pip fails. I'll stop investigating this for now and try to make CI pass.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557154643 https://github.com/pydata/xarray/pull/3557#issuecomment-557154643 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE1NDY0Mw== keewis 14808389 2019-11-21T16:08:53Z 2019-11-21T16:26:32Z MEMBER

I'm currently trying to move everything except python and iris into the pip section, let's see what kind of memory usage that gives us.

Edit: that almost worked, it failed in the execution state. I'll try to install iris from github.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557153710 https://github.com/pydata/xarray/pull/3557#issuecomment-557153710 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE1MzcxMA== dcherian 2448579 2019-11-21T16:06:53Z 2019-11-21T16:06:53Z MEMBER

How about we pip install iris master just to see if we can get things to work?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557148330 https://github.com/pydata/xarray/pull/3557#issuecomment-557148330 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE0ODMzMA== keewis 14808389 2019-11-21T15:55:26Z 2019-11-21T15:55:26Z MEMBER

probably, but the only issue I found is this: https://pypi.org/project/iris/, which means iris is only available on conda-forge?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557143891 https://github.com/pydata/xarray/pull/3557#issuecomment-557143891 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE0Mzg5MQ== dcherian 2448579 2019-11-21T15:46:25Z 2019-11-21T15:46:25Z MEMBER

maybe pip is the solution to all of these problems

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557143391 https://github.com/pydata/xarray/pull/3557#issuecomment-557143391 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE0MzM5MQ== keewis 14808389 2019-11-21T15:45:22Z 2019-11-21T15:45:22Z MEMBER

and now I ran into the shadowing issue we had in #3199. I think the culprit is cfgrib which uses extras_require to pull in xarray. Is there any way to tell conda not to use that? We could also try to pull it using pip, since that works (at least for me) and bypasses the download of the packaged xarray.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557142852 https://github.com/pydata/xarray/pull/3557#issuecomment-557142852 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzE0Mjg1Mg== dcherian 2448579 2019-11-21T15:44:14Z 2019-11-21T15:44:14Z MEMBER

Does anyhting on this page help? It's likely we have requested extra resources for xarray.

https://docs.readthedocs.io/en/stable/guides/build-using-too-many-resources.html#use-pip-when-possible

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
557111157 https://github.com/pydata/xarray/pull/3557#issuecomment-557111157 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NzExMTE1Nw== keewis 14808389 2019-11-21T14:33:07Z 2019-11-21T14:33:07Z MEMBER

okay, now the cherry-picking is done and the number of touched files is down to a more reasonable 14, though I dropped the non-PR commits. I'm not sure which of these was really necessary, but I guess that's something that we will notice once we try to get CI and readthedocs to work?

The original branch is still available for comparison on scipy19-docs

@dcherian (or anyone else who knows a bit more about RTD than me), can you help me fix the readthedocs builds? I can't test anything as long as the environment creation fails.

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
556897055 https://github.com/pydata/xarray/pull/3557#issuecomment-556897055 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1Njg5NzA1NQ== rabernat 1197350 2019-11-21T03:11:56Z 2019-11-21T03:11:56Z MEMBER

Sorry for creating this mess! 🤦‍♂️ Thank you for trying to clean it up.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
556750283 https://github.com/pydata/xarray/pull/3557#issuecomment-556750283 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1Njc1MDI4Mw== dcherian 2448579 2019-11-21T01:16:08Z 2019-11-21T01:16:08Z MEMBER

:+1:

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
556713265 https://github.com/pydata/xarray/pull/3557#issuecomment-556713265 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NjcxMzI2NQ== keewis 14808389 2019-11-21T00:58:46Z 2019-11-21T00:58:46Z MEMBER

it looks like it would be easier to do this by rebasing / cherry-picking onto master: there would not be as many conflicts as using merge would have (look at the list: 146 changed files). I'll cherry-pick and force push afterwards.

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373
556552883 https://github.com/pydata/xarray/pull/3557#issuecomment-556552883 https://api.github.com/repos/pydata/xarray/issues/3557 MDEyOklzc3VlQ29tbWVudDU1NjU1Mjg4Mw== keewis 14808389 2019-11-20T23:48:01Z 2019-11-20T23:48:01Z MEMBER

something that could in the long run reduce the memory usage might be conda-metachannel (blog post), but I think the integration into conda is in its early stages?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   merge scipy19 docs 526273373

Advanced export

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

CSV options: 

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);