home / github / pull_requests

Menu
  • Search all tables
  • GraphQL API

pull_requests: 288875953

This data as json

id node_id number state locked title user body created_at updated_at closed_at merged_at merge_commit_sha assignee milestone draft head base author_association auto_merge repo url merged_by
288875953 MDExOlB1bGxSZXF1ZXN0Mjg4ODc1OTUz 3028 closed 0 Add "errors" keyword argument to drop() and drop_dims() (#2994) 5852283 <!-- Feel free to remove check-list items aren't relevant to your change --> - [x] Closes #2994 - [x] Tests added - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API This addresses #2994 by adding an "errors" keyword argument to `Dataset.drop()`, `Dataset.drop_dims()`, and `DataArray.drop()`. I stuck with pandas' convention of using either `errors='raise'`, now the default that maintains previous behavior by raising an error if any passed label is not found in the dataset/array, or `errors='ignore'` in which case any missing labels are silently ignored. This seems like a pretty straightforward change; mainly it is just skipping checks for missing labels when `errors == 'ignore'` and passing the errors keyword over to the pandas method when using `index.drop()`. Hopefully there are no subtleties that I've missed. I added documentation to the appropriate methods, although I have been struggling to build the docs locally and am unsure if they look right. Also this is my first attempt to contribute to any project, so suggestions and feedback are welcome. 2019-06-17T14:34:19Z 2019-06-20T15:48:00Z 2019-06-20T15:48:00Z 2019-06-20T15:48:00Z 9c0bbf744a5235b4187f87de49175e6776d813cb     0 51a22fa4bcc0904de63966cce24314806f0d4462 c2a2a6efcaf2d279c78da4ba3a87ea96afe78be0 CONTRIBUTOR   13221727 https://github.com/pydata/xarray/pull/3028  

Links from other tables

  • 0 rows from pull_requests_id in labels_pull_requests
Powered by Datasette ยท Queries took 0.729ms