issue_comments

Table actions

8 rows where issue = 349077990 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 3

author_association 2

issue 1

  • New Resample-Syntax leading to cancellation of dimensions · 8 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
542257276 https://github.com/pydata/xarray/issues/2356#issuecomment-542257276 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDU0MjI1NzI3Ng== dcherian 2448579 2019-10-15T15:01:33Z 2019-10-15T15:01:33Z MEMBER

Fixed in 0.13.0

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412382654 https://github.com/pydata/xarray/issues/2356#issuecomment-412382654 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjM4MjY1NA== fujiisoup 6815844 2018-08-13T00:32:02Z 2018-08-13T00:32:02Z MEMBER

Thanks, @dcherian. It looks the original API was designed mainly for 1d arrays, and documentation does not describe clearly how to apply them to multi-dimensional arrays.

I split this issue into two, #2362 and #2363.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412331569 https://github.com/pydata/xarray/issues/2356#issuecomment-412331569 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjMzMTU2OQ== dcherian 2448579 2018-08-12T09:58:09Z 2018-08-12T09:58:09Z MEMBER

Oh sorry, i was mistaken. Looks like pandas does not require the repeated dimension. https://pandas.pydata.org/pandas-docs/stable/timeseries.html#resampling

The current API works like groupby but i don't think that was the original intent (#1269 #1272). 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412249701 https://github.com/pydata/xarray/issues/2356#issuecomment-412249701 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjI0OTcwMQ== fujiisoup 6815844 2018-08-11T04:22:06Z 2018-08-11T04:22:06Z MEMBER

BTW, is this API (repeated dimension names) intended? I am slightly wondering that this is a little different from the rolling counterpart. In rolling, we do ds.rolling(time=3).sum(), we do not need to (cannot) specify the dimension name in sum.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412249612 https://github.com/pydata/xarray/issues/2356#issuecomment-412249612 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjI0OTYxMg== fujiisoup 6815844 2018-08-11T04:19:40Z 2018-08-11T04:19:40Z MEMBER

Is the argument time='M' only mean to be freqency='M'?

It means resampling with frequency='M' *along coordinate named 'time'.

what would be the syntax of your command, if the time dimension has the name 'TIMES'?

It should be python data["TOT_PREC"].resample(TIMES="M").sum(dim='TIMES')

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412109805 https://github.com/pydata/xarray/issues/2356#issuecomment-412109805 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjEwOTgwNQ== dcherian 2448579 2018-08-10T15:02:00Z 2018-08-10T15:02:00Z MEMBER

The repeated dimension follows pandas syntax. It's nice because the syntax is similar to the usual reduction DataArray.sum().

.resample(TIMES='M').sum(dim='TIMES') should work as long as TIMES is datetime64.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
412056121 https://github.com/pydata/xarray/issues/2356#issuecomment-412056121 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMjA1NjEyMQ== rpnaut 30219501 2018-08-10T11:30:45Z 2018-08-10T11:30:45Z NONE

Thank you @dcherian . Do you think, that two times giving the dimension time as argument is useful?

OR MAYBE i understand everything wrong: Is the argument time='M' only mean to be freqency='M'? And the name for the time dimension is now given by the argument "dim"? Or let me ask the question different: what would be the syntax of your command, if the time dimension has the name 'TIMES'?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990
411923961 https://github.com/pydata/xarray/issues/2356#issuecomment-411923961 https://api.github.com/repos/pydata/xarray/issues/2356 MDEyOklzc3VlQ29tbWVudDQxMTkyMzk2MQ== dcherian 2448579 2018-08-09T22:57:06Z 2018-08-09T22:57:06Z MEMBER

datamonth = data["TOT_PREC"].resample(time="M").sum(dim='time') should do what you want.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   New Resample-Syntax leading to cancellation of dimensions 349077990

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]);