docs(todo): COMPCOV ffmpeg A/B — null coverage, corpus bloat - #30
Merged
Merged
Conversation
12 seeds x 2k iters, L0/L2 + L0 control, corpus replayed without COMPCOV. Real edges 5W/7L p=0.77; L2 corpus +70% (12W/0L) for the same coverage; ~5% eps cost. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Qc7F1N9YjmuzLAxiE1XJhp
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the COMPCOV review-leftovers TODO entry with a stock-ffmpeg A/B showing no measurable real-coverage benefit, approximately 70% corpus bloat and a nonsignificant ~5% throughput cost for Layer 2, using in-process replay to exclude synthetic marks from the coverage metric. Flow diagram for the COMPCOV ffmpeg A/B measurementflowchart LR
L0[L0 run] --> C0[Final corpus]
L2[L2 run] --> C2[Final corpus]
C0 --> R[In-process replay with COMPCOV unset]
C2 --> R
R --> M[Count distinct real edge ids]
M --> O[Compare coverage, corpus size, and throughput]
CTRL[L0 control run] --> CCTRL[Control corpus]
CCTRL --> R
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
daedalus
marked this pull request as ready for review
September 27, 2026 16:15
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Correct the contradictory claim, FFmpeg metric values, and corpus-efficiency ratios.
Review effort: Lite
Findings: 2
Open (2)
What changed in this PR
Documents FFmpeg COMPCOV A/B results, including coverage, corpus size, and throughput findings.
Changes:
- Adds benchmark measurements to
docs/TODO.md. - Records Layer 2’s corpus growth and lack of measurable coverage gain.
| File | Description |
|---|---|
docs/TODO.md |
Documents FFmpeg COMPCOV A/B findings. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Records the COMPCOV A/B on ffmpeg in
docs/TODO.md. Only the TODO text changes; no code.Setup: stock
ffmpeg_read_nosan.sofromtools/build_ffmpeg_ready.sh. Its shim is built with cmplog and has 100k const-cmp sites, so both halves of COMPCOV run. Arms are L0, L2, and a second L0 run as a noise control, each over 12 seeds × 2k iterations. Metric: each final corpus is replayed in-process with COMPCOV unset, and the distinct real edge ids are counted.🤖 Generated with Claude Code
https://claude.ai/code/session_01Qc7F1N9YjmuzLAxiE1XJhp
Generated by Claude Code
Summary by Sourcery
Record the ffmpeg COMPCOV A/B results and its coverage, corpus, and throughput implications in the review TODO.
Enhancements:
Documentation: