Add sequence-backed container infrastructure - #977
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## codex/inline-containers-03-sequence-benchmarks #977 +/- ##
==================================================================================
- Coverage 87.65% 87.49% -0.17%
==================================================================================
Files 97 98 +1
Lines 36491 37520 +1029
==================================================================================
+ Hits 31987 32828 +841
- Misses 4504 4692 +188 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will degrade performance by 3.17%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | tests[eggcc-extraction] |
1.4 s | 1.6 s | -11.64% |
| ⚡ | Simulation | tests[eggcc-2mm] |
8.7 s | 8.2 s | +6.11% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing codex/inline-containers-04-infrastructure (9ee921c) with codex/inline-containers-03-sequence-benchmarks (fd5fa6f)
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. ↩
Summary
SequenceTablerow containing[canonical key..., identity].Stack
Draft, stacked on #976. This is PR 4 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