issues

Table actions

5 rows where type = "pull" and user = 950575 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

type 1

  • pull · 5 ✖

state 1

repo 1

id node_id number title user state locked assignee milestone comments created_at updated_at ▲ closed_at author_association active_lock_reason draft pull_request body reactions performed_via_github_app state_reason repo type
320283034 MDExOlB1bGxSZXF1ZXN0MTg1OTg5ODY1 2105 Deprecate decode timedelta ocefpaf 950575 closed 0     3 2018-05-04T13:50:33Z 2019-05-17T13:48:30Z 2019-05-17T13:48:30Z CONTRIBUTOR   0 pydata/xarray/pulls/2105
  • [X] Closes #1621 (remove if there is no corresponding issue, which should only be the case for minor changes)
  • [X] Tests added (for all bug fixes or enhancements)
  • [X] Tests passed (for all non-documentation changes)
  • [X] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

I'll add tests, docs, and the whats-new entry later if I'm on the right path here.

xref.: #843, #940, and #2085

See http://nbviewer.jupyter.org/gist/ocefpaf/e736c07faf3d4c9361ecf546a692c2cd

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2105/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
170698635 MDExOlB1bGxSZXF1ZXN0ODA5OTU3NDE= 962 Two minor docs fixes ocefpaf 950575 closed 0     0 2016-08-11T17:18:28Z 2016-08-11T21:40:47Z 2016-08-11T21:40:46Z CONTRIBUTOR   0 pydata/xarray/pulls/962

~~I am not sure how to build the docs locally (yet) to test these changes.~~

Edit: I found the env file in the docs and tested the docs locally. These changes looks fine in the HTML.

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/962/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
169276671 MDExOlB1bGxSZXF1ZXN0ODAwMDg4NTU= 940 Don't convert time data to timedelta by default ocefpaf 950575 closed 0     6 2016-08-04T02:19:36Z 2016-08-11T16:15:05Z 2016-08-11T16:15:05Z CONTRIBUTOR   0 pydata/xarray/pulls/940

I don't really like this PR... Too much change for a simple thing. I may try again soon.

Closes #843 

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/940/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
153126324 MDExOlB1bGxSZXF1ZXN0Njg5NDAxMDI= 844 Add a filter_by_attrs method to Dataset ocefpaf 950575 closed 0     27 2016-05-04T22:08:07Z 2016-08-03T17:53:43Z 2016-08-03T17:53:42Z CONTRIBUTOR   0 pydata/xarray/pulls/844

This PR adds the get_variables_by_attributes method similar to the one in netCDF4-python and netcdf-java libraries. It is useful to filter a Dataset to known/expected attributes.

@shoyer I don't really like the docs nor the change log entry I created. I will look at them again tomorrow with fresh eyes to see if I can improve them.

Closes https://github.com/pydata/xarray/issues/567

xref: https://github.com/pydata/xarray/issues/567#issuecomment-216947679

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/844/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
152888663 MDExOlB1bGxSZXF1ZXN0Njg3ODU0NzM= 842 Fix #665 decode_cf_timedelta 2D ocefpaf 950575 closed 0     8 2016-05-03T22:26:34Z 2016-05-14T00:36:12Z 2016-05-04T17:11:59Z CONTRIBUTOR   0 pydata/xarray/pulls/842

Long time listener first time caller :wink:

I am not 100% about this PR though. I think that there are cases when we need the actual data rather than the timedelta. In this notebook we have wave period ('mper') that should be seconds ranging 0-30 and not those big numpy timedelta numbers.

I know that I can get that behavior with decode_times=True when opening the dataset, but then the time coordinate get decoded as well. (Maybe I am way off and there is a way to do this.)

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/842/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options: 

CREATE TABLE [issues] (
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [number] INTEGER,
   [title] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [state] TEXT,
   [locked] INTEGER,
   [assignee] INTEGER REFERENCES [users]([id]),
   [milestone] INTEGER REFERENCES [milestones]([id]),
   [comments] INTEGER,
   [created_at] TEXT,
   [updated_at] TEXT,
   [closed_at] TEXT,
   [author_association] TEXT,
   [active_lock_reason] TEXT,
   [draft] INTEGER,
   [pull_request] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [state_reason] TEXT,
   [repo] INTEGER REFERENCES [repos]([id]),
   [type] TEXT
);
CREATE INDEX [idx_issues_repo]
    ON [issues] ([repo]);
CREATE INDEX [idx_issues_milestone]
    ON [issues] ([milestone]);
CREATE INDEX [idx_issues_assignee]
    ON [issues] ([assignee]);
CREATE INDEX [idx_issues_user]
    ON [issues] ([user]);