issue_comments

Table actions

5 rows where issue = 512564022 and user = 14808389 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 1

  • keewis · 5 ✖

issue 1

  • tests for datasets with units · 5 ✖

author_association 1

id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
552104640 https://github.com/pydata/xarray/pull/3447#issuecomment-552104640 https://api.github.com/repos/pydata/xarray/issues/3447 MDEyOklzc3VlQ29tbWVudDU1MjEwNDY0MA== keewis 14808389 2019-11-09T14:17:44Z 2019-11-09T14:17:44Z MEMBER

I think we need to review most if not all tests once we try to make them pass, right now there are too many errors (mostly from pint) to know if they actually do what they should.

 
{
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   tests for datasets with units 512564022
552096649 https://github.com/pydata/xarray/pull/3447#issuecomment-552096649 https://api.github.com/repos/pydata/xarray/issues/3447 MDEyOklzc3VlQ29tbWVudDU1MjA5NjY0OQ== keewis 14808389 2019-11-09T12:43:17Z 2019-11-09T12:43:17Z MEMBER

thanks, @max-sixty

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   tests for datasets with units 512564022
551933045 https://github.com/pydata/xarray/pull/3447#issuecomment-551933045 https://api.github.com/repos/pydata/xarray/issues/3447 MDEyOklzc3VlQ29tbWVudDU1MTkzMzA0NQ== keewis 14808389 2019-11-08T18:10:37Z 2019-11-08T18:10:37Z MEMBER

Is there anything left to do before this is ready to merge? I'd like to use some of the changes to the helper functions made here in #3493.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   tests for datasets with units 512564022
548113449 https://github.com/pydata/xarray/pull/3447#issuecomment-548113449 https://api.github.com/repos/pydata/xarray/issues/3447 MDEyOklzc3VlQ29tbWVudDU0ODExMzQ0OQ== keewis 14808389 2019-10-30T21:09:17Z 2019-10-30T21:09:17Z MEMBER

I think they at least partially test the same code: if two DataArray objects that get passed to the Dataset constructor as data variables have the same dimensions / coordinates, they get merged. I think that happens even if the values of the dims / coords are equal.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   tests for datasets with units 512564022
548096875 https://github.com/pydata/xarray/pull/3447#issuecomment-548096875 https://api.github.com/repos/pydata/xarray/issues/3447 MDEyOklzc3VlQ29tbWVudDU0ODA5Njg3NQ== keewis 14808389 2019-10-30T20:25:11Z 2019-10-30T20:25:11Z MEMBER

I think this is ready for review and merge. I am especially interested in comments on Dataset.__init__ and Dataset.test_merge, since at the moment they seem to test the same code.

There is still a lot to do, and I will also have to go over all tests (DataArray and Dataset) to remove any obvious mistakes (I found a lot from the code in #3238), but if there are any that are not so obvious, they probably won't be found until someone works on making the tests pass.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   tests for datasets with units 512564022

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