Skip to content

DatasetView.map fix keep_attrs #10219

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 15, 2025

Conversation

mathause
Copy link
Collaborator

@mathause mathause commented Apr 11, 2025

Allow setting keep_attrs in DatasetView.map as per the comment in the method. I copied the logic over from xr.Dataset.map.

@github-actions github-actions bot added topic-documentation topic-DataTree Related to the implementation of a DataTree class labels Apr 11, 2025
@mathause mathause marked this pull request as ready for review April 11, 2025 11:43
@mathause
Copy link
Collaborator Author

Failures look unrelated.

@mathause mathause requested a review from TomNicholas April 11, 2025 13:00
@mathause mathause changed the title DatasetView.map fix keep_attrs DatasetView.map fix keep_attrs Apr 11, 2025
@max-sixty
Copy link
Collaborator

lgtm

would love to get to a default of keep_attrs for everything and remove these kwargs!

@mathause mathause added the plan to merge Final call for comments label Apr 14, 2025
@mathause mathause merged commit 430d642 into pydata:main Apr 15, 2025
32 of 38 checks passed
dcherian added a commit that referenced this pull request Apr 27, 2025
* main: (76 commits)
  Update how-to-add-new-backend.rst (#10240)
  Support extension array indexes (#9671)
  Switch documentation to pydata-sphinx-theme (#8708)
  Bump codecov/codecov-action from 5.4.0 to 5.4.2 in the actions group (#10239)
  Fix mypy, min-versions CI, xfail Zarr tests (#10255)
  Remove `test_dask_layers_and_dependencies` (#10242)
  Fix: Docs generation create temporary files that are not cleaned up. (#10238)
  opendap / dap4 support for pydap backend (#10182)
  Add RangeIndex (#10076)
  Fix mypy (#10232)
  Fix doctests (#10230)
  Fix broken Sphinx Roles (#10225)
  `DatasetView.map` fix `keep_attrs` (#10219)
  Add datatree repr asv (#10214)
  CI: Automatic PR labelling is back (#10201)
  Fixes dimension order in `xarray.Dataset.to_stacked_array` (#10205)
  Fix references to core classes in docs (#10207)
  Update pre-commit hooks (#10208)
  add `scipy-stubs` as extra `[types]` dependency (#10202)
  Fix sparse dask repr test (#10200)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan to merge Final call for comments topic-DataTree Related to the implementation of a DataTree class topic-documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants