home / github

Menu
  • Search all tables
  • GraphQL API

issue_comments

Table actions
  • GraphQL API for issue_comments

4 rows where issue = 411365882 and user = 14808389 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

These facets timed out: author_association, issue

user 1

  • keewis · 4 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
841508030 https://github.com/pydata/xarray/issues/2773#issuecomment-841508030 https://api.github.com/repos/pydata/xarray/issues/2773 MDEyOklzc3VlQ29tbWVudDg0MTUwODAzMA== keewis 14808389 2021-05-14T21:16:04Z 2021-05-14T21:16:04Z MEMBER

it might be better to not special-case the "units" attribute, so this should be fixed by _repr_inline_.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature request: show units in dataset overview 411365882
670015358 https://github.com/pydata/xarray/issues/2773#issuecomment-670015358 https://api.github.com/repos/pydata/xarray/issues/2773 MDEyOklzc3VlQ29tbWVudDY3MDAxNTM1OA== keewis 14808389 2020-08-06T15:54:38Z 2020-08-06T16:36:28Z MEMBER

4248 pushes the formatting to duck arrays, so unit-aware arrays like pint can provide a function to properly format itself. If we still want to format units attributes, we'd need to reopen this.

Edit: reopened it until we have a decision about formatting the units attributes

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature request: show units in dataset overview 411365882
662110153 https://github.com/pydata/xarray/issues/2773#issuecomment-662110153 https://api.github.com/repos/pydata/xarray/issues/2773 MDEyOklzc3VlQ29tbWVudDY2MjExMDE1Mw== keewis 14808389 2020-07-21T21:14:46Z 2020-07-21T21:15:31Z MEMBER

Instead of trying to come up with our own formatting, how about supporting a _repr_short_(self, length) method on the duck array (with a fall back to the current behavior)? That way duck arrays have to explicitly define the format (or have a compatibility package like pint-xarray provide it for them) if they want something different from their normal repr and we don't have to add duck array specific code.

This won't help with displaying the units attributes (which we don't really need once we have support for pint arrays in indexes).

{
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature request: show units in dataset overview 411365882
565518795 https://github.com/pydata/xarray/issues/2773#issuecomment-565518795 https://api.github.com/repos/pydata/xarray/issues/2773 MDEyOklzc3VlQ29tbWVudDU2NTUxODc5NQ== keewis 14808389 2019-12-13T16:57:36Z 2019-12-13T16:57:36Z MEMBER

it was argued in pint that the unit is part of the data, so we should keep it as close to the data as possible. How about <xarray.Dataset> Dimensions: (time: 3, x: 988, y: 822) Coordinates: * x (x) [m] float64 ... * y (y) [m] float64 ... * time (time) [s] datetime64[ns] ... Data variables: rainfall (time, y, x) [mm] float32 ... max_temp (time, y, x) [deg C] float32 ... or <xarray.Dataset> Dimensions: (time: 3, x: 988, y: 822) Coordinates: * x (x) float64 [m] ... * y (y) float64 [m] ... * time (time) datetime64[ns] [s] ... Data variables: rainfall (time, y, x) float32 [mm] ... max_temp (time, y, x) float32 [deg C] ... The issue with the second example is that it is easy to confuse with numpy's dtype, though. Maybe we should use parentheses instead?

re special casing: I think would be fine for attributes since we already special case them for plotting, but I don't know about duck arrays. Even if we want to special case them, there are many unit libraries with different interfaces so we would either need to special case all of them or require a specific interface (or a function to retrieve the necessary data?).

Also, we should keep in mind is that using more horizontal space for the units results in less space for data. And we should not forget about https://github.com/dask/dask/issues/5329#issue-485927396, where a different kind of format was proposed, at least for the values of a DataArray.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Feature request: show units in dataset overview 411365882

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