Merged
Merged
Conversation
Merging this PR will improve performance by 2.68%
⚡ 10 improved benchmarks
❌ 2 regressed benchmarks
✅ 127 untouched benchmarks
⏩ 135 skipped benchmarks1
Warning
Please fix the performance issues or acknowledge them on CodSpeed.
Performance Changes
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | test_pickle_load[arm] |
1.6 ms | 1.8 ms | -9.41% |
| ❌ | Simulation | test_decode_type[arm-msgpack-False] |
803.1 µs | 840.5 µs | -4.46% |
| ⚡ | Simulation | test_decode_type[arm-msgpack-None] |
748 µs | 676.5 µs | +10.57% |
| ⚡ | Simulation | test_decode_type[arm-json-True] |
938.5 µs | 866.3 µs | +8.34% |
| ⚡ | Simulation | test_decode_type[arm-json-enum] |
1.2 ms | 1.1 ms | +6.41% |
| ⚡ | Simulation | test_decode_type[arm-msgpack-float] |
2.2 ms | 2.1 ms | +4.31% |
| ⚡ | Simulation | test_decode_type[arm-msgpack-int] |
2.3 ms | 2.2 ms | +3.73% |
| ⚡ | Simulation | test_decode_convert_type_non_strict[arm-msgpack-float] |
2.2 ms | 2.2 ms | +3.43% |
| ⚡ | Simulation | test_decode_type[arm-json-int] |
2.6 ms | 2.5 ms | +3.07% |
| ⚡ | Simulation | test_encode_type[arm-json-datetime] |
1.7 ms | 1.7 ms | +2.95% |
| ⚡ | Simulation | test_decode_convert_type_non_strict[arm-json-float] |
3.2 ms | 3.1 ms | +2.47% |
| ⚡ | Simulation | test_decode_type[arm-json-float] |
3.2 ms | 3.1 ms | +2.3% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ofek/pyemscripten ( 542a9a5 ) with main ( 551e2dc )
Footnotes
-
135 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. ↩
| return lk->table[index]; | ||
| /* Compute the dense-table index in 64-bit space so large offsets | ||
| * cannot wrap through Py_ssize_t on 32-bit builds. */ | ||
| if (key >= lk->offset) { |
ofek
mentioned this pull request
Merged
Labels
None yet