home / github

Menu
  • GraphQL API
  • Search all tables

issue_comments

Table actions
  • GraphQL API for issue_comments

13 rows where user = 199813 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

issue 8

  • Feature Request: Hierarchical storage and processing in xarray 3
  • Starter property-based test suite 2
  • Fix matplotlib in upstream-dev test config 2
  • fix matplotlib errors for single level discrete colormaps 2
  • Complete renaming xray -> xarray 1
  • sparse and other duck array issues 1
  • failing upstream-dev CI: matplotlib 1
  • Allow the label run-upstream to run upstream CI 1

user 1

  • tacaswell · 13 ✖

author_association 1

  • NONE 13
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1528096647 https://github.com/pydata/xarray/pull/7787#issuecomment-1528096647 https://api.github.com/repos/pydata/xarray/issues/7787 IC_kwDOAMm_X85bFOeH tacaswell 199813 2023-04-28T21:07:24Z 2023-04-28T21:07:24Z NONE

I'm also relatively sure that if you are willing to put a floor on the version of Matplotlib you support get_window_extent will use it's internally cached renderer (and when we make it uniformly optional we also fixed the cache invalidation logic).

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Allow the label run-upstream to run upstream CI 1684281101
1059499222 https://github.com/pydata/xarray/issues/4118#issuecomment-1059499222 https://api.github.com/repos/pydata/xarray/issues/4118 IC_kwDOAMm_X84_JqzW tacaswell 199813 2022-03-04T20:25:47Z 2022-03-04T20:25:47Z NONE

@LunarLanding You may also be interested in awkward array.

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature Request: Hierarchical storage and processing in xarray 628719058
875121115 https://github.com/pydata/xarray/issues/4118#issuecomment-875121115 https://api.github.com/repos/pydata/xarray/issues/4118 MDEyOklzc3VlQ29tbWVudDg3NTEyMTExNQ== tacaswell 199813 2021-07-06T22:21:02Z 2021-07-06T22:21:02Z NONE

That sounds right to me -- a downside of tags is that they can't be (uniquely) expressed in a hierarchical arrangement like those found in HDF5/netCDF4 files.

hdf5 allows for internal links so a datasets and groups can appear in multiple places in the tree. You can even make cycles where groups are in them selves (or their children). The NeXuS format (the xray/neutron one) makes heavy use of this to let data appear both where it "makes sense" from a science point of view from an instrumentation point of view.

I think it is reasonable to expect that netcdf -> xarray -> netcdf always , however I think it is unreasonable to ask that xarray -> netcdf -> xarray will always work. I think it is OK if xarray can express more complex relationship and structures that you can in netcdf (or hdf5 or any existing at-rest format). In an extreme case, consider an interface to a database that returns xarrays 😈 .

{
    "total_count": 4,
    "+1": 4,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature Request: Hierarchical storage and processing in xarray 628719058
802863863 https://github.com/pydata/xarray/issues/4118#issuecomment-802863863 https://api.github.com/repos/pydata/xarray/issues/4118 MDEyOklzc3VlQ29tbWVudDgwMjg2Mzg2Mw== tacaswell 199813 2021-03-19T14:14:13Z 2021-03-19T14:14:13Z NONE

This is related to some very recent work we have been doing at NSLS-II, primarily lead by @danielballan .

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature Request: Hierarchical storage and processing in xarray 628719058
780798734 https://github.com/pydata/xarray/issues/3245#issuecomment-780798734 https://api.github.com/repos/pydata/xarray/issues/3245 MDEyOklzc3VlQ29tbWVudDc4MDc5ODczNA== tacaswell 199813 2021-02-17T19:33:05Z 2021-02-17T19:33:05Z NONE

Is there any motion on this? This come up in relation to akward array today (see https://github.com/scikit-hep/awkward-1.0/issues/749 ).

If xarray goes with the to_numpy spelling as well then all of {pandas, xarray, akwardarray} have the same spelling which is nice from the Matplotlib side of things.

I think the as_ vs to_ distinction the "to_" makes more sense as it gets you out of xarray land and drops you back to raw numpy land.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  sparse and other duck array issues 484240082
663174724 https://github.com/pydata/xarray/pull/4256#issuecomment-663174724 https://api.github.com/repos/pydata/xarray/issues/4256 MDEyOklzc3VlQ29tbWVudDY2MzE3NDcyNA== tacaswell 199813 2020-07-23T18:53:13Z 2020-07-23T18:53:13Z NONE

Can you point me at the configs that upload to the correct place?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  fix matplotlib errors for single level discrete colormaps 664363493
663033165 https://github.com/pydata/xarray/pull/4256#issuecomment-663033165 https://api.github.com/repos/pydata/xarray/issues/4256 MDEyOklzc3VlQ29tbWVudDY2MzAzMzE2NQ== tacaswell 199813 2020-07-23T14:17:07Z 2020-07-23T14:17:07Z NONE

It does look like the rackspace server we were pushing to seems to be down. This is more of a @matthew-brett question.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  fix matplotlib errors for single level discrete colormaps 664363493
663030222 https://github.com/pydata/xarray/issues/4226#issuecomment-663030222 https://api.github.com/repos/pydata/xarray/issues/4226 MDEyOklzc3VlQ29tbWVudDY2MzAzMDIyMg== tacaswell 199813 2020-07-23T14:11:33Z 2020-07-23T14:11:33Z NONE

That changes was definitely intentional in BoundaryNorm, but we thought that we were giving a clear error in a case where you used to get a divide by 0 error.

I think we used to implicitly change 1 boundary into a degenerate 2 boundary, I can see the argument that we should be more forgiving, but in boundry norm I'm not sure what that would mean (internally we are going to make every value to either over or under. If you want to support this at the xarray level you should do the [4] -> [4, 4] conversion which will work with all version of Matplotlib.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  failing upstream-dev CI: matplotlib 657230088
643016806 https://github.com/pydata/xarray/issues/4145#issuecomment-643016806 https://api.github.com/repos/pydata/xarray/issues/4145 MDEyOklzc3VlQ29tbWVudDY0MzAxNjgwNg== tacaswell 199813 2020-06-12T01:37:32Z 2020-06-12T01:37:32Z NONE

Thanks to @QuLogic the wheels are building and uploading again.

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Fix matplotlib in upstream-dev test config 636665269
642375798 https://github.com/pydata/xarray/issues/4145#issuecomment-642375798 https://api.github.com/repos/pydata/xarray/issues/4145 MDEyOklzc3VlQ29tbWVudDY0MjM3NTc5OA== tacaswell 199813 2020-06-11T02:59:05Z 2020-06-11T02:59:05Z NONE

Those wheels are (I think) built from https://github.com/MacPython/matplotlib-wheels/tree/master . We also use that system to build the wheels we push to pypi, but from the build-X.Y branch. The details of how the travis file needed to be configured changed and we never merged those changes back to master (which we have now done). but the builds are failing with an obscure error ( https://travis-ci.org/github/MacPython/matplotlib-wheels/jobs/697078308 ).

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Fix matplotlib in upstream-dev test config 636665269
371819126 https://github.com/pydata/xarray/pull/1972#issuecomment-371819126 https://api.github.com/repos/pydata/xarray/issues/1972 MDEyOklzc3VlQ29tbWVudDM3MTgxOTEyNg== tacaswell 199813 2018-03-09T13:58:15Z 2018-03-09T13:58:15Z NONE

Set the backend to Agg on travis as you don't have a xserever running. You probably want to manually force a draw as well.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Starter property-based test suite 303103716
371681439 https://github.com/pydata/xarray/pull/1972#issuecomment-371681439 https://api.github.com/repos/pydata/xarray/issues/1972 MDEyOklzc3VlQ29tbWVudDM3MTY4MTQzOQ== tacaswell 199813 2018-03-09T01:20:44Z 2018-03-09T01:20:44Z NONE

What do the failing data sets look like? Does it get easier or harder to find failures if you go up to 10x10? What sort of exceptions are you getting?

You can shove a fair amount of configuration in to pytest.ini to make these opt-in.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Starter property-based test suite 303103716
169216487 https://github.com/pydata/xarray/issues/704#issuecomment-169216487 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTIxNjQ4Nw== tacaswell 199813 2016-01-06T04:21:12Z 2016-01-06T04:21:12Z NONE

@shoyer Thank you :beers:

import xarray as xy ?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Complete renaming xray -> xarray 124867009

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]);
Powered by Datasette · Queries took 966.255ms · About: xarray-datasette