home / github

Menu
  • GraphQL API
  • Search all tables

issue_comments

Table actions
  • GraphQL API for issue_comments

10 rows where issue = 327725925 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

  • philippjfr 5
  • rabernat 2
  • shoyer 2
  • fmaussion 1

author_association 2

  • MEMBER 5
  • NONE 5

issue 1

  • HoloViews based plotting API · 10 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
401603966 https://github.com/pydata/xarray/issues/2199#issuecomment-401603966 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDQwMTYwMzk2Ng== philippjfr 1550771 2018-07-01T12:35:56Z 2018-07-01T12:35:56Z NONE

Thanks for everyone's feedback, due to trademark concerns we decided to rename both the library and the API to .hvplot. There should be a release and an announcement in the coming week.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
394179490 https://github.com/pydata/xarray/issues/2199#issuecomment-394179490 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5NDE3OTQ5MA== philippjfr 1550771 2018-06-03T17:56:40Z 2018-06-03T17:56:40Z NONE

Thanks again for the feedback, I've decided to go with .holoplot in the end. I'll work on finishing some of geo related features today and get a 0.1 release and announcement out this week.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393764489 https://github.com/pydata/xarray/issues/2199#issuecomment-393764489 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5Mzc2NDQ4OQ== shoyer 1217238 2018-06-01T05:36:11Z 2018-06-01T05:36:11Z MEMBER

I'm not strongly opposed to something like DataArray.hvplot for the accessor, it's just slightly less obvious than DataArray.holoplot.

hv would probably be too short for a good name (but of course this is totally up to you), especially because I can imagine people using hv for a variables name, which can also be accessed via attributes.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393309581 https://github.com/pydata/xarray/issues/2199#issuecomment-393309581 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzMwOTU4MQ== philippjfr 1550771 2018-05-30T20:34:39Z 2018-05-30T20:37:24Z NONE

something like DataArray.hv.plot.contourf() seems too deeply nested.

Actually I suppose that's not what it would be, it could be da.hv.plot and da.hv.contourf with .plot figuring out the kind for you. I quite like that too.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393300042 https://github.com/pydata/xarray/issues/2199#issuecomment-393300042 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzMwMDA0Mg== philippjfr 1550771 2018-05-30T20:06:29Z 2018-05-30T20:16:45Z NONE

I agree the accessor is the best option for now, but I have no strong opinions about the name of the accessor.

Okay thanks, given xarray's preference for accessor names to match projects I'm now leaning toward da.holoplot().

Automatic generation of DynamicMaps. Say I have a DataArray with dimensions ('time', 'lat', 'lon'); I should be able to say da.hv.plot(kdims=['lat', 'lon'] and have time become a dynamic selector.

HoloPlot explicitly does not deal with kdims and vdims instead more closely following the API of pd.DataFrame.plot and xr.DataArray. That said coordinates that are not assigned to the x/y axes will automatically result in a DynamicMap, so this will give you an image plot + a widget to select the time:

python da.holoplot(x='lon', y='lat', kind='image')

To go along with the above, lazy loading of dask-backed arrays

That should happen automatically.

Intelligent faceting which automatically links the facet kdims

You can facet in a number of ways:

python da.isel(time=slice(0, 3)).holoplot(x='lon', y='lat', kind='image', by='time')

will produce three subplots which are linked on the x- and y-axis, i.e. zooming on one will zoom on all unless you set shared_axes=False. You can also generate a grid with:

python da.isel(time=slice(0, 3)).holoplot(x='lon', y='lat', kind='image', row='time', col='some_other_coord')

Plotting not just of DataArrays but Datasets.

This is also already supported, the API here is:

python ds.holoplot(x='lon', y='lat', z=['air', 'surface'])

Will provide a widget to select between the 'air' and 'surface' data variable.

Options for projections, coastlines, etc. associated with geoviews

Currently working on that, it's basically just waiting on new HoloViews/GeoViews releases. The API here is as follows:

python air_ds.air.holoplot.quadmesh( 'lon', 'lat', ['air', 'some_other_variable'], crs=ccrs.PlateCarree(), projection=ccrs.Orthographic(-80, 30), global_extent=True, width=600, height=500, cmap='viridis' ) * gv.feature.coastline

{
    "total_count": 7,
    "+1": 4,
    "-1": 0,
    "laugh": 0,
    "hooray": 3,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393292090 https://github.com/pydata/xarray/issues/2199#issuecomment-393292090 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzI5MjA5MA== rabernat 1197350 2018-05-30T19:42:43Z 2018-05-30T19:42:43Z MEMBER

Oh and another big 👍 to the datashader integration. This is crucial for my datasets.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393291809 https://github.com/pydata/xarray/issues/2199#issuecomment-393291809 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzI5MTgwOQ== rabernat 1197350 2018-05-30T19:41:56Z 2018-05-30T19:41:56Z MEMBER

I am a big fan of holoviews and have been using it extensively for my own work in recent months. So obviously I am a big 👍 on this integration.

I agree the accessor is the best option for now, but I have no strong opinions about the name of the accessor.

Some features I would like to see are things that go beyond the plotting capabilities associated with the matplotlib engine. For example:

  • Automatic generation of DynamicMaps. Say I have a DataArray with dimensions ('time', 'lat', 'lon'); I should be able to say da.hv.plot(kdims=['lat', 'lon'] and have time become a dynamic selector.
  • To go along with the above, lazy loading of dask-backed arrays
  • Intelligent faceting which automatically links the facet kdims
  • Plotting not just of DataArrays but Datasets. The variable itself could become a dynamic selector in a dropdown menu. Basically, I just want to say ds.hv.plot() and have holoviews provide all the options I need to explore the dataset interactively. Kind of like how ncview works. At that point, we won't need ncview anymore.
  • Options for projections, coastlines, etc. associated with geoviews
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393284049 https://github.com/pydata/xarray/issues/2199#issuecomment-393284049 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzI4NDA0OQ== philippjfr 1550771 2018-05-30T19:16:32Z 2018-05-30T19:16:32Z NONE

Thanks for the feedback! I'll try to drive the pandas conversation along, but since I doubt that will be resolved in the near term so I think until then we should definitely pursue the accessor approach (which is much better than the property monkey patching we're doing now).

Personally I'd prefer DataArray.hvplot() since I think even the two extra characters make a difference and something like DataArray.hv.plot.contourf() seems too deeply nested. That said if "our preference for accessor names to match projects" is a solidly established convention I'll defer to that and go with DataArray.holoplot().

@rabernat Since you have used HoloViews with xarray in the past I'd very appreciate your input as well.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393214964 https://github.com/pydata/xarray/issues/2199#issuecomment-393214964 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzIxNDk2NA== shoyer 1217238 2018-05-30T15:53:06Z 2018-05-30T15:53:06Z MEMBER

Very cool! I also think this would be a good use case for a new accessor, perhaps DataArray.holoplot() mirroring our preference for accessor names to match projects.

An engine keyword/option could also be viable, but would require more coordination (e.g., figuring out the plotting interface, which seems to have stalled that plotting issue). That said, if pandas figured out a way to do this I'm sure we would be happy to copy it.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925
393164716 https://github.com/pydata/xarray/issues/2199#issuecomment-393164716 https://api.github.com/repos/pydata/xarray/issues/2199 MDEyOklzc3VlQ29tbWVudDM5MzE2NDcxNg== fmaussion 10050469 2018-05-30T13:37:41Z 2018-05-30T13:37:41Z MEMBER

It looks like a good use case for accessors. The syntax could then be: DataArray.hv.plot() and would give you full flexibility.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  HoloViews based plotting API 327725925

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