Print benchmark paths relative to the repo root - #21
Merged
Merged
Conversation
bench/data/run.log is committed as the provenance record for the CSVs beside it, and BENCHMARKS.md points at those CSVs as the source of truth. Two of its lines carried absolute paths, so a public repo advertised the operator's home directory and username, and the transcript read as something that could only have been produced on one particular laptop. The log now names build/bench/bench and bench/data, which is what a reader needs and all they need. run_matrix.sh, which writes those lines, strips the repo root from any path under it before printing, so the next sweep does not reintroduce what this commit removes. A BIN pointed somewhere outside the tree still prints in full, because there the absolute path is the honest answer. Nothing reads run.log, so no consumer changes. The log is kept rather than deleted because it is the only record of the sweep that produced the committed CSVs.
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.
bench/data/run.logis tracked, and it recorded absolute paths from the machine that produced it, so a public repo carried the author's home directory layout on lines 2 and 454.This changes
run_matrix.shto print paths relative to the repo root, and rewrites the two committed lines to match, so the log stays readable and the next run does not reintroduce it.🤖 Generated with Claude Code