Repository navigation
Test IO file sizes in Rust and array sizes in Python doctests - #9792
robert3005 wants to merge 2 commits into
Conversation
e1cf626 to
8bb4130
Compare
Merging this PR will improve performance by 10.74%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | WallTime | words_gather_dispatch_neon[65536] |
2.3 µs | 2 µs | +10.74% |
Tip
Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/python-io-doctest-size (9a58a45) with develop (f234bc6)
Footnotes
-
518 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. ↩
8bb4130 to
d6147dd
Compare
joseph-isaacs
left a comment
There was a problem hiding this comment.
Can we move this into a rust test on the written file size. It really useful to know when we change file size
|
Here you go, there's a rust test now, I kept the python change since this is useful for docs |
e8208a8 to
c0aae71
Compare
|
This PR has been marked as stale because it has been open for 14 days with no activity. Please comment or remove the stale label if you wish to keep it active, otherwise it will be closed in 7 days |
c0aae71 to
db43544
Compare
Signed-off-by: Robert Kruszewski <github@robertk.io>
Signed-off-by: Robert Kruszewski <github@robertk.io>
db43544 to
9a58a45
Compare
Python io doc tests check compressed size in bytes since that's what they're talking about. Keep the file size test as a rust test