Repository navigation
Conversation
…166 examples/githits_ab.py and examples/pilot_rehearsal.py come back with the tests that covered them (tests/test_pilot_rehearsal.py and the Rehearsal case in tests/test_experiment_githits_api.py). docs/EXPERIMENTS.md, docs/GITHITS.md, docs/PAID_PROBLEM.md and docs/ROADMAP_STATUS.md still point at both. The counted GitHits pilot runs from the pinned commit b298cb6, which already has both files. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Admission
Class: R (reversible/internal)
Agent-platform: claude
Reviewed-by: none recorded
Admitted-by: pending, held as a draft until Mateo has looked at it
Evidence: tested
Objective
Bring back
examples/githits_ab.pyandexamples/pilot_rehearsal.py, which #166 removed, ahead of the GitHits pilot runs.Before: main had no GitHits A/B rehearsal or pilot rehearsal, while docs/EXPERIMENTS.md, docs/GITHITS.md, docs/PAID_PROBLEM.md and docs/ROADMAP_STATUS.md still told readers to run them.
After: both scripts are back on main with the tests that covered them, and the docs point at files that exist again.
How: the two scripts and
tests/test_pilot_rehearsal.pyare checked out from the parent of #166, and theRehearsalcase (with its imports) is put back intests/test_experiment_githits_api.pyby reversing that file's hunk from #166. Nothing else in #166 is reverted.What changed
examples/githits_ab.py,examples/pilot_rehearsal.py: restored as they were before Delete code no deployed app reaches: harnesses, examples, jev_api, unused symbols #166tests/test_pilot_rehearsal.py: restoredtests/test_experiment_githits_api.py: theRehearsalcase andload_rehearsalrestoredWhat did not change
experiments/githits-pilot/pilot.json), which already has both files. This PR does not change the pin.Evidence
The full suite did not finish inside a 10 minute local limit; CI runs it on this PR.
Known limitations
tests/durations.jsonhas no entries for the restored tests, so the shard runner uses the median for them.Generated by Claude Code