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

Improve aggregate(), sum() #72

Merged
merged 19 commits into from Sep 27, 2022
Merged

Improve aggregate(), sum() #72

merged 19 commits into from Sep 27, 2022

Conversation

khaeru
Copy link
Owner

@khaeru khaeru commented Sep 20, 2022

  • Add new computations assign_units(), convert_units()
  • Use pd.Series.groupby(), instead of .stack() in AttrSeries.sum() to handle larger data without memory issues.
  • Remove support for Python 3.7.

PR checklist

  • Checks all ✅
  • Update documentation
  • Update doc/whatsnew.rst

@khaeru khaeru added the enh New feature or request label Sep 20, 2022
@khaeru khaeru self-assigned this Sep 20, 2022
- Argument N_data allows specifying arbitrary length.
- Allow up to 11 dimensions.
- Use categoricals to improve performance in get_large_quantity().
- Duplicate indices.
- Large data.

Both fail with the v1.13.0 implementation using .unstack()
Tolerates larger data and duplicate indices.
These are alternatives to apply_units with simpler behaviour.
@khaeru khaeru merged commit 0f557c3 into main Sep 27, 2022
@khaeru khaeru deleted the enh/2022-W38 branch September 27, 2022 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enh New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant