Repository navigation
bench/sweep.py: per-commit output dirs; WFA2-lib's new location - #46
Merged
Merged
Conversation
cjfields
changed the title
Results now default to target/bench-sweep/<host>-<commit>[-dirty][-quick]/ so runs of different commits on one host (A/B tests) don't overwrite each other. "-dirty" marks modified tracked files. The help for --no-build notes that existing builds may be from another commit than the label. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The C library moved from smarco/WFA2-lib to Marco-Sola-Lab/WFA2-lib. Update the clone URLs (CI, build_c.sh, bench/sweep.py, README) and the links and issue references in docs, rustdoc and tests. The pinned commit 3282f83 is fetchable from the new repository. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
cjfields
force-pushed
the
bench/out-per-commit
branch
from
October 15, 2026 21:21
d9f2b6b to
53a475c
Compare
Open
11 of 18 tasks
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
1 participant
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.
Small follow-up to #44, for A/B runs (#31).
bench/sweep.pynow writes totarget/bench-sweep/<host>-<commit>[-dirty][-quick]/by default, so runs of different commits on one host don't overwrite each other.--outstill overrides it.-dirtymarks modified tracked files; untracked files don't count.rust commitline insummary.mduses the same label.--no-buildhelp now notes that existing builds may be from a different commit than the label. That happened in testing: the per-host binary had been built from another branch.Tested with
--quick --no-buildon macOS.WFA2-lib's new location (folded in from #47)
The C library moved from
smarco/WFA2-libtoMarco-Sola-Lab/WFA2-lib.tests/differential/build_c.sh(default source),bench/sweep.py(offline hint), README..claude/docs, rustdoc insrc/aligner.rs, comments intests/corrected_endsfree.rs. GitHub redirects the old names, but these now match the repository.3282f83can be fetched from the new repository, so CI's clone-then-checkout keeps working.Existing local
./WFA2-libcheckouts still have the oldoriginURL, which redirects. To update one:git -C WFA2-lib remote set-url origin https://github.com/Marco-Sola-Lab/WFA2-lib.🤖 Generated with Claude Code