home / github

Menu
  • GraphQL API
  • Search all tables

issue_comments

Table actions
  • GraphQL API for issue_comments

22 rows where author_association = "CONTRIBUTOR" and user = 731499 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 11

  • Guess the complementary dimension when only one is passed to pcolormesh 4
  • simple groupby_bins 10x slower than numpy 4
  • Fixes OS error arising from too many files open 3
  • pass projection argument to plt.subplot when faceting with cartopy transform 2
  • transfer projection to implied subplots when faceting 2
  • confusing dataset creation process 2
  • missing return value in sample function calls (I think) 1
  • way to apply functions to subplots when faceting 1
  • numpy function very slow on DataArray compared to DataArray.values 1
  • open_mfdataset crashes when files are present with datasets of dimension = 0 1
  • let the user specify figure dpi when plotting 1

user 1

  • vnoel · 22 ✖

author_association 1

  • CONTRIBUTOR · 22 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1177245770 https://github.com/pydata/xarray/issues/6758#issuecomment-1177245770 https://api.github.com/repos/pydata/xarray/issues/6758 IC_kwDOAMm_X85GK1hK vnoel 731499 2022-07-07T08:26:26Z 2022-07-07T08:26:26Z CONTRIBUTOR

@dcherian Just to be complete, I thought the following one-liner would work as well:

sums, x = np.histogram(latitude, bins, weights=array)

but apparently it produces slightly different results for reasons I don't understand

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  simple groupby_bins 10x slower than numpy 1295939038
1177163992 https://github.com/pydata/xarray/issues/6758#issuecomment-1177163992 https://api.github.com/repos/pydata/xarray/issues/6758 IC_kwDOAMm_X85GKhjY vnoel 731499 2022-07-07T06:53:52Z 2022-07-07T06:53:52Z CONTRIBUTOR

the IDL histogram function but in numpy.

Apparently not as awesome!

Yeah, the present solution is less general, but most of the time I'm just counting stuff, and this is much faster than what I was doing, so I'm happy ;-)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  simple groupby_bins 10x slower than numpy 1295939038
1176777842 https://github.com/pydata/xarray/issues/6758#issuecomment-1176777842 https://api.github.com/repos/pydata/xarray/issues/6758 IC_kwDOAMm_X85GJDRy vnoel 731499 2022-07-06T21:40:37Z 2022-07-06T21:40:37Z CONTRIBUTOR

@dcherian I just tested your numpy suggestions, and I'm getting 100x speedups compared to my naive numpy approach (~200µs vs ~20ms). Thankyouthankyouthankyou! I've been doing this for years, I can't believe I've never run into that particular solution. It's like the IDL histogram function but in numpy. I'm going to use this like crazy Thanks again

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  simple groupby_bins 10x slower than numpy 1295939038
1176701867 https://github.com/pydata/xarray/issues/6758#issuecomment-1176701867 https://api.github.com/repos/pydata/xarray/issues/6758 IC_kwDOAMm_X85GIwur vnoel 731499 2022-07-06T20:37:12Z 2022-07-06T20:37:12Z CONTRIBUTOR

@dcherian this means that xarray's groupby_bins will always be slow unless flox is installed, correct? I have unfortunately little or no say on what packages are installed on the system that runs my code.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  simple groupby_bins 10x slower than numpy 1295939038
394771642 https://github.com/pydata/xarray/issues/2216#issuecomment-394771642 https://api.github.com/repos/pydata/xarray/issues/2216 MDEyOklzc3VlQ29tbWVudDM5NDc3MTY0Mg== vnoel 731499 2018-06-05T16:19:38Z 2018-06-05T16:19:38Z CONTRIBUTOR

Apparently not. https://stackoverflow.com/questions/48227792/how-to-change-dpi-of-my-pandas-dataframe-plot

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  let the user specify figure dpi when plotting 329483009
371970858 https://github.com/pydata/xarray/issues/1719#issuecomment-371970858 https://api.github.com/repos/pydata/xarray/issues/1719 MDEyOklzc3VlQ29tbWVudDM3MTk3MDg1OA== vnoel 731499 2018-03-09T23:06:20Z 2018-03-09T23:06:20Z CONTRIBUTOR

Has this behavior been fixed by the change to NetCDF4 ? https://github.com/Unidata/netcdf4-python/issues/743

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  open_mfdataset crashes when files are present with datasets of dimension = 0 274298111
287099757 https://github.com/pydata/xarray/pull/1198#issuecomment-287099757 https://api.github.com/repos/pydata/xarray/issues/1198 MDEyOklzc3VlQ29tbWVudDI4NzA5OTc1Nw== vnoel 731499 2017-03-16T15:46:14Z 2017-03-16T15:46:14Z CONTRIBUTOR

with that keyword the open_mfdataset goes through ! my code crashes later because the data is too big to fit in memory, but that's another problem ;-)

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Fixes OS error arising from too many files open 199900056
287058651 https://github.com/pydata/xarray/pull/1198#issuecomment-287058651 https://api.github.com/repos/pydata/xarray/issues/1198 MDEyOklzc3VlQ29tbWVudDI4NzA1ODY1MQ== vnoel 731499 2017-03-16T13:37:05Z 2017-03-16T13:37:05Z CONTRIBUTOR

Hey @pwolfram, I installed your branch and tried to open_mfdataset on 1474 hdf5 files.

I got the following: File "/users/noel/.conda/envs/python3/lib/python3.6/site-packages/xarray/backends/api.py", line 524, in open_mfdataset File "/users/noel/.conda/envs/python3/lib/python3.6/site-packages/xarray/backends/api.py", line 524, in <listcomp> File "/users/noel/.conda/envs/python3/lib/python3.6/site-packages/xarray/backends/api.py", line 299, in open_dataset File "/users/noel/.conda/envs/python3/lib/python3.6/site-packages/xarray/backends/netCDF4_.py", line 203, in __init__ File "/users/noel/.conda/envs/python3/lib/python3.6/site-packages/xarray/backends/netCDF4_.py", line 178, in _open_netcdf4_group File "netCDF4/_netCDF4.pyx", line 1848, in netCDF4._netCDF4.Dataset.__init__ (netCDF4/_netCDF4.c:13992) OSError: Too many open files

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Fixes OS error arising from too many files open 199900056
284435076 https://github.com/pydata/xarray/pull/1291#issuecomment-284435076 https://api.github.com/repos/pydata/xarray/issues/1291 MDEyOklzc3VlQ29tbWVudDI4NDQzNTA3Ng== vnoel 731499 2017-03-06T15:46:23Z 2017-03-06T15:46:23Z CONTRIBUTOR

I hope everything's ok now. Thanks for the hand-holding

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Guess the complementary dimension when only one is passed to pcolormesh 211391408
283930759 https://github.com/pydata/xarray/pull/1291#issuecomment-283930759 https://api.github.com/repos/pydata/xarray/issues/1291 MDEyOklzc3VlQ29tbWVudDI4MzkzMDc1OQ== vnoel 731499 2017-03-03T11:25:06Z 2017-03-03T11:25:06Z CONTRIBUTOR

Can anyone knowledgeable tell me how to fix the appveyor build? It's complaining of a failing test_dask_distributed_integration_test...

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Guess the complementary dimension when only one is passed to pcolormesh 211391408
283694163 https://github.com/pydata/xarray/pull/1291#issuecomment-283694163 https://api.github.com/repos/pydata/xarray/issues/1291 MDEyOklzc3VlQ29tbWVudDI4MzY5NDE2Mw== vnoel 731499 2017-03-02T15:59:49Z 2017-03-02T16:51:10Z CONTRIBUTOR

thanks ! Do we need to worry about the "appveyor" build fails? The failure seems to happen somewhere unrelated

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Guess the complementary dimension when only one is passed to pcolormesh 211391408
283682931 https://github.com/pydata/xarray/pull/1291#issuecomment-283682931 https://api.github.com/repos/pydata/xarray/issues/1291 MDEyOklzc3VlQ29tbWVudDI4MzY4MjkzMQ== vnoel 731499 2017-03-02T15:22:03Z 2017-03-02T15:22:03Z CONTRIBUTOR

I have added a couple of exceptions to the _infer_xy_labels function and updated the tests as best I could

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Guess the complementary dimension when only one is passed to pcolormesh 211391408
277969098 https://github.com/pydata/xarray/issues/1249#issuecomment-277969098 https://api.github.com/repos/pydata/xarray/issues/1249 MDEyOklzc3VlQ29tbWVudDI3Nzk2OTA5OA== vnoel 731499 2017-02-07T11:14:42Z 2017-02-07T11:14:42Z CONTRIBUTOR

ok, I see. In case there's a vote, I vote for raising an error in 1D too :-)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  confusing dataset creation process 205414496
277767832 https://github.com/pydata/xarray/issues/1249#issuecomment-277767832 https://api.github.com/repos/pydata/xarray/issues/1249 MDEyOklzc3VlQ29tbWVudDI3Nzc2NzgzMg== vnoel 731499 2017-02-06T18:22:08Z 2017-02-06T18:22:08Z CONTRIBUTOR

I don't understand the "raise an error" option. Would it mean that creating a Dataset with 1D arrays with no named dimension would raise an error? That would be very impractical.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  confusing dataset creation process 205414496
277336569 https://github.com/pydata/xarray/issues/1247#issuecomment-277336569 https://api.github.com/repos/pydata/xarray/issues/1247 MDEyOklzc3VlQ29tbWVudDI3NzMzNjU2OQ== vnoel 731499 2017-02-03T19:17:09Z 2017-02-03T19:17:09Z CONTRIBUTOR

you mean, within numpy?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  numpy function very slow on DataArray compared to DataArray.values 205215815
277276242 https://github.com/pydata/xarray/pull/1198#issuecomment-277276242 https://api.github.com/repos/pydata/xarray/issues/1198 MDEyOklzc3VlQ29tbWVudDI3NzI3NjI0Mg== vnoel 731499 2017-02-03T15:27:04Z 2017-02-03T15:27:04Z CONTRIBUTOR

I'm just chiming in to signify my interest in seeing this issue solved. I have just hit "OSError: Too many open files". The data itself is not even huge, but it's scattered across many files and it's a PITA to revert to manual concatenation -- I've grown used to dask doing the work for me ;-)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Fixes OS error arising from too many files open 199900056
272415578 https://github.com/pydata/xarray/pull/1205#issuecomment-272415578 https://api.github.com/repos/pydata/xarray/issues/1205 MDEyOklzc3VlQ29tbWVudDI3MjQxNTU3OA== vnoel 731499 2017-01-13T10:52:32Z 2017-01-13T10:52:32Z CONTRIBUTOR

As it is, the xarray code becomes tied up with cartopy. If cartopy somehow changes its syntax (e.g. by renaming the transform kw) this code won't work.

Maybe there's another, more abstract way to do this?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  transfer projection to implied subplots when faceting 200593854
272414593 https://github.com/pydata/xarray/pull/1205#issuecomment-272414593 https://api.github.com/repos/pydata/xarray/issues/1205 MDEyOklzc3VlQ29tbWVudDI3MjQxNDU5Mw== vnoel 731499 2017-01-13T10:47:28Z 2017-01-13T10:47:28Z CONTRIBUTOR

One concern I have is that it is conceivable that one might pass a transform keyword to plot() intended to interact with another library that hijacks matplotlib (i.e. not cartopy). In that case, matplotlib will get an unintended projection argument. That is a corner case though.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  transfer projection to implied subplots when faceting 200593854
272409178 https://github.com/pydata/xarray/issues/1201#issuecomment-272409178 https://api.github.com/repos/pydata/xarray/issues/1201 MDEyOklzc3VlQ29tbWVudDI3MjQwOTE3OA== vnoel 731499 2017-01-13T10:19:08Z 2017-01-13T10:19:08Z CONTRIBUTOR

I don't understand what you mean by 'adding you own to the signature'. Adding my own what to what signature?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  pass projection argument to plt.subplot when faceting with cartopy transform 200364693
272395987 https://github.com/pydata/xarray/issues/1201#issuecomment-272395987 https://api.github.com/repos/pydata/xarray/issues/1201 MDEyOklzc3VlQ29tbWVudDI3MjM5NTk4Nw== vnoel 731499 2017-01-13T09:14:30Z 2017-01-13T09:14:30Z CONTRIBUTOR

I am confused.

I planned to identify the plotting function that accepted the transform keyword (used in the code sample above), and add its value in subplot_kws if it wasn't already.

However it looks like no xarray code contains the transform keyword. However I definitely can use it in calls to plot(). I don't understand what is going on. Can someone enlighten me?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  pass projection argument to plt.subplot when faceting with cartopy transform 200364693
272173661 https://github.com/pydata/xarray/issues/1202#issuecomment-272173661 https://api.github.com/repos/pydata/xarray/issues/1202 MDEyOklzc3VlQ29tbWVudDI3MjE3MzY2MQ== vnoel 731499 2017-01-12T14:16:29Z 2017-01-12T14:16:29Z CONTRIBUTOR

I have found in issue #604 a example of how to do this. I have submitted in issue #1203 a change to the docs.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  way to apply functions to subplots when faceting 200369077
258854045 https://github.com/pydata/xarray/pull/1088#issuecomment-258854045 https://api.github.com/repos/pydata/xarray/issues/1088 MDEyOklzc3VlQ29tbWVudDI1ODg1NDA0NQ== vnoel 731499 2016-11-07T14:45:24Z 2016-11-07T14:45:24Z CONTRIBUTOR

I was briefly confused by the examples as at first they seemed to me to do nothing -- read data and forget it. This change prevents any similar confusion :-)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  missing return value in sample function calls (I think) 187661575

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 17.339ms · About: xarray-datasette