Skip to content

Remove more scalar_at from sparse arrays#7439

Open
gatesn wants to merge 3 commits intodevelopfrom
ngates/more-scalar-at
Open

Remove more scalar_at from sparse arrays#7439
gatesn wants to merge 3 commits intodevelopfrom
ngates/more-scalar-at

Conversation

@gatesn
Copy link
Copy Markdown
Contributor

@gatesn gatesn commented Apr 15, 2026

Benchmark case Before (mean) After (mean) Speedup
extend_from_array_non_zctl_overlapping (1000, 8) 1.413 ms 80.71 us 17.5x
extend_from_array_non_zctl_overlapping (1000, 32) 2.586 ms 83.67 us 30.9x
extend_from_array_non_zctl_overlapping (10000, 8) 9.708 ms 560.3 us 17.3x
extend_from_array_zctl (1000, 8) 1.116 ms 14.31 us 78.0x
extend_from_array_zctl (1000, 64) 5.511 ms 38.68 us 142.5x
extend_from_array_zctl (10000, 8) 11.01 ms 108.4 us 101.6x

Before = commit d72bf9b93 (pre-fix), After = commit c4ad4e2ad (the ListViewBuilder scalar-at removal fix).

gatesn added 2 commits April 14, 2026 20:16
Signed-off-by: Nicholas Gates <nick@nickgates.com>
Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn requested a review from robert3005 April 15, 2026 00:16
@gatesn gatesn enabled auto-merge (squash) April 15, 2026 00:16
@gatesn gatesn added the changelog/performance A performance improvement label Apr 15, 2026
robert3005
robert3005 previously approved these changes Apr 15, 2026
Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005 robert3005 disabled auto-merge April 15, 2026 08:30
@robert3005 robert3005 dismissed their stale review April 15, 2026 08:31

The benchmark numbers are not for this benchmark

@robert3005
Copy link
Copy Markdown
Contributor

I pushed a commit to fix the build. Benchmarks need to be trimmed, even on my machine they take ~10ms which is way too slow for microbenchmark imho. The improvement here is more modest 35-50%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/performance A performance improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants