Skip to content
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

Fix bug where we had extra groups in expected_groups. #112

Merged
merged 2 commits into from
Jun 23, 2022

Conversation

dcherian
Copy link
Collaborator

This affected _factorize_multiple.

Closes #111

Thanks to @LunarLanding for reporting

This affected _factorize_multiple.

Closes #111
* fix dask case

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@dcherian dcherian merged commit 5e0b793 into main Jun 23, 2022
@dcherian dcherian deleted the fix-extra-expected-groups branch June 23, 2022 22:44
dcherian added a commit that referenced this pull request Oct 17, 2022
* main: (44 commits)
  Rename repo to xarray-contrib/flox (#125)
  Propagate attrs better (#124)
  Update README.md (#123)
  Update README.md
  micromamba in CI (#122)
  Many optimizations (#120)
  Add xarray groupby test (#121)
  Update README.md
  Update README.md
  Add NASA Xarray grant
  [pre-commit.ci] pre-commit autoupdate (#118)
  Allow custom numpy aggregations for "blockwise" (#117)
  Don't execute hourly climatology notebook (#98)
  Bump actions/setup-python from 3 to 4 (#116)
  Fix factorizing some more. (#115)
  Correctly factorize values outside bin edges
  Better error message
  Avoid a copy.
  Fix bug where we had extra groups in expected_groups. (#112)
  Fix binning by nD variable
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants