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

Feature: Skip None in Aggregations #789

Merged
merged 7 commits into from
Mar 24, 2025

Conversation

gwaramadze
Copy link
Contributor

@gwaramadze gwaramadze commented Mar 17, 2025

  • Application will not fail if None is encountered in the aggregation context.
  • Additionally, NaN (not a number) will be ignored as well as any calculation with it gives back a useless NaN

@gwaramadze gwaramadze force-pushed the feature/missing-data-in-aggregations branch from d81497d to 1341e65 Compare March 18, 2025 08:01
@gwaramadze gwaramadze force-pushed the feature/missing-data-in-aggregations branch from 0114fd3 to cf2f688 Compare March 18, 2025 08:18
@gwaramadze gwaramadze changed the title Feature: Skip None and NaN in Aggregations Feature: Skip None in Aggregations Mar 18, 2025
tim-quix
tim-quix previously approved these changes Mar 18, 2025
@gwaramadze gwaramadze force-pushed the feature/missing-data-in-aggregations branch from d34a82b to 8555727 Compare March 24, 2025 09:52
@gwaramadze gwaramadze merged commit 8151ce4 into main Mar 24, 2025
3 checks passed
@gwaramadze gwaramadze deleted the feature/missing-data-in-aggregations branch March 24, 2025 16:33
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.

4 participants