Skip to content

explicitly cast the dtype of where's condition parameter to bool #10087

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 15 commits into from
Mar 13, 2025

Conversation

keewis
Copy link
Collaborator

@keewis keewis commented Mar 2, 2025

(this is necessary because array-api-strict raises an error otherwise)

dcherian added a commit to dcherian/xarray that referenced this pull request Mar 7, 2025
dcherian added a commit that referenced this pull request Mar 7, 2025
@keewis
Copy link
Collaborator Author

keewis commented Mar 12, 2025

contrary to the way CI looks it appears the last few commits fixed the remaining issues (the remaining failing tests are unrelated). So this should finally be ready for merging.

cc @dcherian, @eni-awowale

@dcherian dcherian added the plan to merge Final call for comments label Mar 13, 2025
@keewis keewis enabled auto-merge (squash) March 13, 2025 19:40
@keewis keewis merged commit bc33fda into pydata:main Mar 13, 2025
36 checks passed
@keewis keewis deleted the where-dtype branch March 13, 2025 19:42
dcherian added a commit to dcherian/xarray that referenced this pull request Mar 19, 2025
* main: (67 commits)
  Refactor datetime and timedelta encoding for increased robustness (pydata#9498)
  Fix test_distributed::test_async (pydata#10138)
  Refactor concat / combine / merge into `xarray/structure` (pydata#10134)
  Split `apply_ufunc` out of `computation.py` (pydata#10133)
  Refactor modules from `core` into `xarray.computation` (pydata#10132)
  Refactor compatibility modules into xarray.compat package (pydata#10131)
  Fix type issues from pandas stubs (pydata#10128)
  Don't skip tests when on a `mypy` branch (pydata#10129)
  Change `python_files` in `pyproject.toml` to a list (pydata#10127)
  Better `uv` compatibility (pydata#10124)
  explicitly cast the dtype of `where`'s condition parameter to `bool` (pydata#10087)
  Use `to_numpy` in time decoding (pydata#10081)
  Pin pandas stubs (pydata#10119)
  Fix broken Zarr test (pydata#10109)
  Update asv badge url in README.md (pydata#10113)
  fix and supress some test warnings (pydata#10104)
  Improve handling of dtype and NaT when encoding/decoding masked and packaged datetimes and timedeltas (pydata#10050)
  Ensure KeyError raised for zarr datasets missing dim names (pydata#10025)
  Add typos check to pre-commit hooks (pydata#10040)
  Bump codecov/codecov-action from 5.3.1 to 5.4.0 in the actions group (pydata#10090)
  ...
dcherian added a commit to dcherian/xarray that referenced this pull request Mar 19, 2025
* main: (85 commits)
  Adds open_datatree and load_datatree to the tutorial module (pydata#10082)
  Fix version in requires_zarr_v3 fixture (pydata#10145)
  Fix `open_datatree` when `decode_cf=False` (pydata#10141)
  [docs] `DataTree` cannot be constructed from `DataArray` (pydata#10142)
  Refactor datetime and timedelta encoding for increased robustness (pydata#9498)
  Fix test_distributed::test_async (pydata#10138)
  Refactor concat / combine / merge into `xarray/structure` (pydata#10134)
  Split `apply_ufunc` out of `computation.py` (pydata#10133)
  Refactor modules from `core` into `xarray.computation` (pydata#10132)
  Refactor compatibility modules into xarray.compat package (pydata#10131)
  Fix type issues from pandas stubs (pydata#10128)
  Don't skip tests when on a `mypy` branch (pydata#10129)
  Change `python_files` in `pyproject.toml` to a list (pydata#10127)
  Better `uv` compatibility (pydata#10124)
  explicitly cast the dtype of `where`'s condition parameter to `bool` (pydata#10087)
  Use `to_numpy` in time decoding (pydata#10081)
  Pin pandas stubs (pydata#10119)
  Fix broken Zarr test (pydata#10109)
  Update asv badge url in README.md (pydata#10113)
  fix and supress some test warnings (pydata#10104)
  ...
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants