Move Vec containers to sequence storage - #979
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## codex/inline-containers-05-unified-maintenance #979 +/- ##
==================================================================================
+ Coverage 87.57% 87.69% +0.12%
==================================================================================
Files 98 98
Lines 37705 37945 +240
==================================================================================
+ Hits 33019 33275 +256
+ Misses 4686 4670 -16 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will improve performance by 17.68%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | tests[eggcc-extraction] |
1.6 s | 1.3 s | +17.68% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing codex/inline-containers-06-vec (d8a18cf) with codex/inline-containers-05-unified-maintenance (ea2856a)
Footnotes
-
227 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. ↩
3573a3f to
d8a18cf
Compare
Summary
[rebuild flag, elements...].Stack
Draft, stacked on #978. This is PR 6 of 12 in the sequence-backed container stack.
Validation
Validated on the complete 12-commit stack:
cargo nextest run --workspace: 1,362 tests passedcargo test --workspace --doccargo clippy --workspace --all-targets -- -D warningscargo fmt --all -- --check