Improve take on Chunked(FSL) and FSL(Chunked) array alongside implementation of take using PiecewiseSequence by robert3005 · Pull Request #8881 · vortex-data/vortex

Performance Gate Passed

⚡ 13 improved benchmarks
✅ 1828 untouched benchmarks
🆕 6 new benchmarks
⏩ 55 skipped benchmarks¹

	Mode	Benchmark	`BASE`	`HEAD`	Efficiency
⚡	Simulation	`take_chunked_fsl_random[32, 64]`	517.1 µs	166.3 µs	×3.1
⚡	Simulation	`take_chunked_fsl_sorted[32, 64]`	515.2 µs	170.5 µs	×3
⚡	Simulation	`take_fsl_chunked_elements_random[32, 64]`	428.2 µs	152.4 µs	×2.8
⚡	Simulation	`take_fsl_chunked_elements_sorted[32, 64]`	434.5 µs	158.4 µs	×2.7
⚡	Simulation	`take_chunked_fsl_random[16, 64]`	355.6 µs	152.7 µs	×2.3
⚡	Simulation	`take_chunked_fsl_sorted[16, 64]`	350.1 µs	161.7 µs	×2.2
⚡	Simulation	`take_fsl_chunked_elements_random[16, 64]`	283.7 µs	137.3 µs	×2.1
⚡	Simulation	`fsl_large`	89.2 µs	44.5 µs	×2
⚡	Simulation	`take_chunked_fsl_random[8, 64]`	279.6 µs	144 µs	+94.11%
⚡	Simulation	`take_fsl_chunked_elements_sorted[16, 64]`	270.7 µs	147.5 µs	+83.61%
⚡	Simulation	`take_chunked_fsl_sorted[8, 64]`	271 µs	153.6 µs	+76.5%
⚡	Simulation	`take_fsl_chunked_elements_random[8, 64]`	198.4 µs	129.4 µs	+53.33%
⚡	Simulation	`take_fsl_chunked_elements_sorted[8, 64]`	189.5 µs	140.1 µs	+35.28%
🆕	Simulation	`take_chunked_compressed_fsl[16, 64]`	N/A	226.6 µs	N/A
🆕	Simulation	`take_chunked_compressed_fsl[32, 64]`	N/A	242.1 µs	N/A
🆕	Simulation	`take_chunked_compressed_fsl[8, 64]`	N/A	220.2 µs	N/A
🆕	Simulation	`take_chunked_compressed_fsl_piecewise[16, 64]`	N/A	150.4 µs	N/A
🆕	Simulation	`take_chunked_compressed_fsl_piecewise[32, 64]`	N/A	161.3 µs	N/A
🆕	Simulation	`take_chunked_compressed_fsl_piecewise[8, 64]`	N/A	142.4 µs	N/A

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.

_{Comparing rk/chunkedfsltake (11b7ae6) with develop (f3d6795)}

55 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. ↩