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 groupby_bins when labels are specified #7769

Merged
merged 6 commits into from
Apr 20, 2023

Conversation

dcherian
Copy link
Contributor

@gsieros can you try this out please?

xarray/tests/test_groupby.py Outdated Show resolved Hide resolved
xarray/tests/test_groupby.py Outdated Show resolved Hide resolved
xarray/tests/test_groupby.py Outdated Show resolved Hide resolved
xarray/tests/test_groupby.py Outdated Show resolved Hide resolved
xarray/tests/test_groupby.py Outdated Show resolved Hide resolved
@gsieros
Copy link

gsieros commented Apr 20, 2023

Just confirmed that the fix works for the actual application that I was using (where N-NW etc were used as labels in binning wind directions), so all good in that respect.

@dcherian dcherian enabled auto-merge (squash) April 20, 2023 16:38
@dcherian
Copy link
Contributor Author

Thanks @gsieros, and apologies for the trouble. Clearly our test suite was insufficient.

I'll push out a release soon.

@dcherian dcherian merged commit a4c54a3 into pydata:main Apr 20, 2023
dcherian added a commit to dcherian/xarray that referenced this pull request Apr 25, 2023
* main:
  Bump codecov/codecov-action from 3.1.2 to 3.1.3 (pydata#7781)
  Fix whats-new
  [skip-ci] dev whats-new (pydata#7775)
  [skip-ci] Release 2023.04.2 (pydata#7774)
  Fix groupby_bins when labels are specified (pydata#7769)
  Docstrings examples for string methods (pydata#7669)
  Add dev whats-new
  Add benchmark against latest release on main. (pydata#7753)
@dcherian dcherian deleted the fix-groupby-bins branch July 22, 2023 01:01
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.

groupby_bins incorrect correspondence with labels
3 participants