home / github / pull_requests

Menu
  • Search all tables
  • GraphQL API

pull_requests: 380575560

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
380575560 MDExOlB1bGxSZXF1ZXN0MzgwNTc1NTYw 3801 closed 0 Coarsen keep attrs 3376 29958771 <!-- Feel free to remove check-list items aren't relevant to your change --> - [x] Closes #3376 - [x] Tests added - [x] Passes `isort -rc . && black . && mypy . && flake8` - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API I also noticed missing attributes when attempting certain operations, so I have created this pull request to attempt a fix. I modified code in `DataWithCoords.coarsen()` to try applying the same logic used for other methods since #2482 was successfully merged. I added two new tests that adapt the bug reported by @jejjohnson. I believe they should pass when this is fixed. One uses `Dataset.coarsen()` and the other uses `Variable.coarsen`. Both tests currently fail. xarray/tests/test_dataset.py::TestDataset::test_coarsen_keep_attrs - fails because my attempted fix does not retain the attributes. xarray/tests/test_variable.py::TestVariable::test_coarsen_keep_attrs - also fails, but unexpectedly this seems to be because the attributes are _not_ removed by default, or even when explicitly setting the global option to False. 2020-02-27T00:29:13Z 2020-03-06T16:33:11Z 2020-03-02T23:02:56Z 2020-03-02T23:02:55Z 1c5e1cd022a0ff91275c50a50d1c6f88a7abff7d     0 4968c440d439d39dc9b6a068f7e65ad06cf81afe 8512b7bf498c0c300f146447c0b05545842e9404 CONTRIBUTOR   13221727 https://github.com/pydata/xarray/pull/3801  

Links from other tables

  • 0 rows from pull_requests_id in labels_pull_requests
Powered by Datasette · Queries took 0.82ms