Skip to content

Array operations fuzzer operates on smaller arrays#7743

Merged
robert3005 merged 2 commits intodevelopfrom
rk/arrayopsfuzzer
May 1, 2026
Merged

Array operations fuzzer operates on smaller arrays#7743
robert3005 merged 2 commits intodevelopfrom
rk/arrayopsfuzzer

Conversation

@robert3005
Copy link
Copy Markdown
Contributor

I believe some of the reference implementations are very slow so we can't
operate on bigger arrays

Signed-off-by: Robert Kruszewski github@robertk.io

Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005 robert3005 added the changelog/skip Do not list PR in the changelog label Apr 30, 2026
@robert3005 robert3005 enabled auto-merge (squash) April 30, 2026 23:36
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 30, 2026

Merging this PR will degrade performance by 39.88%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 8 improved benchmarks
❌ 8 regressed benchmarks
✅ 1182 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime mix[50%_in/50%_out] 340 µs 393.2 µs -13.54%
WallTime mix[100%_in/0%_out] 500.7 µs 449.5 µs +11.39%
WallTime for[10M_u16] 72.3 µs 120.3 µs -39.88%
WallTime for[10M_u32] 114.6 µs 164.5 µs -30.34%
WallTime dynamic_dispatch_u32[10M] 142.3 µs 94.2 µs +51.09%
WallTime dynamic_dispatch_f32[10M] 114.7 µs 166.9 µs -31.29%
Simulation decompress_rd[f64, (10000, 0.01)] 138.6 µs 121.9 µs +13.7%
Simulation decompress_rd[f32, (100000, 0.01)] 495.1 µs 582.7 µs -15.04%
Simulation decompress_rd[f64, (10000, 0.1)] 138.7 µs 122 µs +13.63%
Simulation decompress_rd[f32, (100000, 0.1)] 495.1 µs 582.7 µs -15.04%
Simulation decompress_rd[f32, (10000, 0.0)] 94.6 µs 85.9 µs +10.14%
Simulation decompress_rd[f64, (100000, 0.01)] 842.6 µs 1,020.5 µs -17.44%
Simulation decompress_rd[f32, (10000, 0.1)] 90.2 µs 82 µs +10%
Simulation decompress_rd[f64, (100000, 0.1)] 842.5 µs 1,020.7 µs -17.46%
Simulation decompress_rd[f64, (10000, 0.0)] 138.5 µs 122.1 µs +13.46%
Simulation decompress_rd[f32, (100000, 0.0)] 583.5 µs 495.7 µs +17.7%

Comparing rk/arrayopsfuzzer (964b3f1) with develop (cb9b138)

Open in CodSpeed

Signed-off-by: Robert Kruszewski <github@robertk.io>
Copy link
Copy Markdown
Contributor

@joseph-isaacs joseph-isaacs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can do this. But we do really want the 16k size going forwards

@robert3005 robert3005 merged commit de36b35 into develop May 1, 2026
62 of 63 checks passed
@robert3005 robert3005 deleted the rk/arrayopsfuzzer branch May 1, 2026 07:42
@robert3005
Copy link
Copy Markdown
Contributor Author

I kept the file fuzzer working on 16k arrays since that looked fine. I think we need to spend some time on reference impls in the fuzzer

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

Labels

changelog/skip Do not list PR in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants