Repository navigation
Audit tests for duplication and coverage - #10458
robert3005 wants to merge 1 commit into
Conversation
70a4472 to
25ba6a5
Compare
Merging this PR will degrade performance by 1.12%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | decode_primitives[i64, (1000, 32)] |
25.4 µs | 37.1 µs | -31.52% |
| ❌ | WallTime | starts_with_avx2[8, VarBin] |
41 µs | 47 µs | -12.72% |
| ❌ | WallTime | bitpack_blocked_compress_avx2 |
6.8 µs | 7.6 µs | -10.28% |
| ⚡ | WallTime | filter_avx2[64] |
2.5 µs | 2.1 µs | +16.46% |
| ⚡ | WallTime | filter_avx2[8] |
1.2 µs | 1 µs | +16.24% |
| ⚡ | WallTime | filter_avx2[80] |
2.6 µs | 2.3 µs | +13.89% |
| ⚡ | WallTime | compare_u8_neon |
2 µs | 1.8 µs | +11.8% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/removeredundanttests (a95bf52) with develop (2664573)
Footnotes
-
43 benchmarks were run, but are now archived. If they were deleted in another branch, consider rebasing to remove them from the report. Instead if they were added back, click here to restore them. ↩
fix the assertions that have been found by test audit extracted from #10458 --------- Signed-off-by: Robert Kruszewski <github@robertk.io>
Signed-off-by: Robert Kruszewski <github@robertk.io>
25ba6a5 to
a95bf52
Compare
Some of the tests covered duplicate logic, they haven't been updated to latest helper functions or simply didn't check the correct thing. This should have audited all of the tests we have in the repo and fix them without losing coverage