Skip to content

Do not rely on np.broadcast_to to perform trivial dimension insertion - #10277

Merged
dcherian merged 5 commits into
pydata:mainfrom
hmaarrfk:broadcast_to
May 6, 2025
Merged

Do not rely on np.broadcast_to to perform trivial dimension insertion#10277
dcherian merged 5 commits into
pydata:mainfrom
hmaarrfk:broadcast_to

Conversation

@hmaarrfk

@hmaarrfk hmaarrfk commented Apr 30, 2025

Copy link
Copy Markdown
Contributor
pytest xarray/tests/test_variable.py::TestVariable::test_set_dims_without_broadcast

hmaarrfk added a commit to hmaarrfk/xarray that referenced this pull request Apr 30, 2025
@hmaarrfk hmaarrfk changed the title Broadcast to Do not rely on np.broadcast_to to perform trivial dimension insertion Apr 30, 2025
Comment thread xarray/tests/test_variable.py
@TomNicholas

Copy link
Copy Markdown
Member

Nice - this would have probably made writing VirtualiZarr easier 😅

@hmaarrfk

hmaarrfk commented May 3, 2025

Copy link
Copy Markdown
Contributor Author

hmm, did i do something to

image

@dcherian
dcherian merged commit b8e3be8 into pydata:main May 6, 2025
@hmaarrfk

hmaarrfk commented May 6, 2025

Copy link
Copy Markdown
Contributor Author

awesome thanks!

dcherian added a commit to dcherian/xarray that referenced this pull request May 7, 2025
* main:
  dev whats-new (pydata#10294)
  Add SeasonGrouper, SeasonResampler (pydata#9524)
  (fix): remove `PandasExtensionArray` from repr (pydata#10291)
  Do not rely on `np.broadcast_to` to perform trivial dimension insertion (pydata#10277)
  DOC: Remove reference to absolufy (pydata#10290)
  Fix for scalar detection (pydata#8821)
  Add Index.validate_dataarray_coord (pydata#10137)
  add redirect for contributing guide (pydata#10282)
  Update pre-commit hooks (pydata#10288)
  Adding xarray-eopf to ecosystem.rst (pydata#10289)
  Add public typing.py module (pydata#10215)
  Rename Twitter to X (pydata#10283)
  Add `xarray-lmfit` extension for curve fitting to ecosystem documentation (pydata#10262)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants