Aggregate Fns

[gatesn]

First PR implementing the Aggregate Functions proposal in vortex-data/rfcs#21

[gatesn]

• AggregateFn MinMax #6724
• Aggregate Fns: Sum #6723
• Aggregate Fns: Mean #6722
• Aggregate Fns #6721 👈 (View in Graphite)
• develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

Merging this PR will improve performance by 22.76%

⚡ 2 improved benchmarks
✅ 392 untouched benchmarks
⏩ 2052 skipped benchmarks¹

Performance Changes

	Mode	Benchmark	BASE	HEAD	Efficiency
⚡	Simulation	take_map[(0.1, 0.5)]	2.6 ms	2.1 ms	+22.76%
⚡	Simulation	take_map[(0.1, 1.0)]	4.2 ms	3.6 ms	+16.84%

---

_{Comparing ngates/aggregate-fns (828f62c) with develop (5d6a3c8)²}

Footnotes

1. 2052 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

2. No successful run was found on develop (e719a17) during the generation of this report, so 5d6a3c8 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

[robert3005]

We shouldn't be forcing upcasting here. Just because you have Decimal it doesn't mean you should do all compute in i256 space