Skip to content

perf(aggr): fuse indexed statistical groups#324

Merged
singaraiona merged 1 commit into
devfrom
agent/indexed-statistical-groups
Jul 10, 2026
Merged

perf(aggr): fuse indexed statistical groups#324
singaraiona merged 1 commit into
devfrom
agent/indexed-statistical-groups

Conversation

@singaraiona

@singaraiona singaraiona commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • extend the grouped-index slice kernel to compute variance/stddev accumulators directly
  • add binary aggregate partials for Pearson correlation, covariance, weighted sum, and weighted average
  • keep eligible statistical aggregate queries on the slice path instead of folding index slices back into a generic selection
  • add indexed/plain and contiguous/non-contiguous parity coverage for every admitted binary aggregate

Why

Filtered group-bys over a grouped symbol index already have exact row slices for each requested key. Statistical and binary aggregate shapes were rejected by the slice kernel, forcing those slices back into a sorted selection and then through generic grouping.

Validation

  • make test: 3,615/3,615 passed under ASan/UBSan
  • clean release build succeeded
  • 14/14 indexed/plain and sorted/unsorted statistical parity checks passed
  • git diff --check passed

@singaraiona singaraiona marked this pull request as ready for review July 10, 2026 04:07
@singaraiona singaraiona merged commit c706079 into dev Jul 10, 2026
10 checks passed
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.

1 participant