home / github

Menu
  • Search all tables
  • GraphQL API

issue_comments

Table actions
  • GraphQL API for issue_comments

8 rows where issue = 332018176 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 5

  • shoyer 2
  • stale[bot] 2
  • rpnaut 2
  • dcherian 1
  • aidanheerdegen 1

author_association 3

  • NONE 4
  • MEMBER 3
  • CONTRIBUTOR 1

issue 1

  • Time bounds returned after an operation with resample-method · 8 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1100969650 https://github.com/pydata/xarray/issues/2231#issuecomment-1100969650 https://api.github.com/repos/pydata/xarray/issues/2231 IC_kwDOAMm_X85Bn3ay stale[bot] 26384082 2022-04-17T23:43:46Z 2022-04-17T23:43:46Z NONE

In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity

If this issue remains relevant, please comment here or remove the stale label; otherwise it will be marked as closed automatically

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
658438489 https://github.com/pydata/xarray/issues/2231#issuecomment-658438489 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDY1ODQzODQ4OQ== dcherian 2448579 2020-07-14T22:09:47Z 2020-07-14T22:09:47Z MEMBER

@rpnaut see the discussion in https://github.com/xarray-contrib/cf-xarray/issues/10 . That discussion focuses more on using the bounds to properly weight points when resampling.

But we could also make .cf.resample generate time bounds and set appropriate attributes on the returned object.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
658429636 https://github.com/pydata/xarray/issues/2231#issuecomment-658429636 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDY1ODQyOTYzNg== rpnaut 30219501 2020-07-14T21:45:39Z 2020-07-14T21:45:39Z NONE

Maybe, I will look to create a wrapper to handle the time_bounds issue for files following the cf-conventions. Note, that not only resample operations should modify the time_bounds, but also the reselect process should take care about time_bounds. As an example, we assume to have in one file A instantenous data (two times a day at 00 UTC and 12 UTC) and in the other file B aggregated data (daily averages with time stamps defined at the end of the aggregation interval). The reselection process of A in B should pick up only the times 12 UTC from file A (or even better: no time steps because aggregation interval in file B is not compatible with instantenous values).

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
656965778 https://github.com/pydata/xarray/issues/2231#issuecomment-656965778 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDY1Njk2NTc3OA== stale[bot] 26384082 2020-07-11T01:55:37Z 2020-07-11T01:55:37Z NONE

In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity

If this issue remains relevant, please comment here or remove the stale label; otherwise it will be marked as closed automatically

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
410950987 https://github.com/pydata/xarray/issues/2231#issuecomment-410950987 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDQxMDk1MDk4Nw== aidanheerdegen 6063709 2018-08-07T06:43:26Z 2018-08-07T06:59:22Z CONTRIBUTOR

Sorry, xarray doesn’t handle time bounds directly, nor does it update metadata according to cfconventions. These were intentional design choices to keep xarray simple, but in principle you could layer cf convention handling on top of xarray.

Nor does it bring along bounds variables when extracting variables from a dataset, e.g. double time(time) ; time:long_name = "time" ; time:cartesian_axis = "T" ; time:calendar_type = "NOLEAP" ; time:bounds = "time_bounds" ; time:units = "days since 0001-01-01" ; time:calendar = "NOLEAP" ; When a variable using the time dimension is extracted from a Dataset, the time_bounds variable is missing.

Is this also an intentional choice or something that xarray could/should support? Or does already and I've missed how to invoke this.

Edit: I've just realised, how is xarray supposed to "bring along" another variable in a DataArray object? I'll leave this query as maybe there is a solution? Have a bounds attribute similar to the coords attribute?

Is this just a dupe of https://github.com/pydata/xarray/issues/1475 ?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
397185013 https://github.com/pydata/xarray/issues/2231#issuecomment-397185013 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDM5NzE4NTAxMw== shoyer 1217238 2018-06-14T06:24:47Z 2018-06-14T06:24:47Z MEMBER

One thing I’ll note is that you probably want to make the time bounds variables coordinates rather than data variables. But that means the time bounds will probably simply be dropped when you resample.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
397167282 https://github.com/pydata/xarray/issues/2231#issuecomment-397167282 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDM5NzE2NzI4Mg== shoyer 1217238 2018-06-14T04:32:25Z 2018-06-14T04:32:25Z MEMBER

Sorry, xarray doesn’t handle time bounds directly, nor does it update metadata according to cfconventions. These were intentional design choices to keep xarray simple, but in principle you could layer cf convention handling on top of xarray.

For this sort of analysis, using a tool like Iris designed to follow cf conventions might make sense. You can convert directly between iris cubes and xarray DataArray objects.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176
396957166 https://github.com/pydata/xarray/issues/2231#issuecomment-396957166 https://api.github.com/repos/pydata/xarray/issues/2231 MDEyOklzc3VlQ29tbWVudDM5Njk1NzE2Ng== rpnaut 30219501 2018-06-13T14:26:53Z 2018-06-13T14:26:53Z NONE

I want to add that sometimes the variable time_bnds is already gone after resampling.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Time bounds returned after an operation with resample-method 332018176

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