Two decisions in this repo are waiting on the same unknown, and one pass over the candidate clones settles both.
The Decisions
How discriminating the "not analyzed" list should be. #59 made inspect_commit.py name every modified file it declined to open, so that silence stops standing in for "looked at and found unchanged". It does not discriminate: a README.md lands in that list beside a data.parquet. That was deliberate, because the alternative is a hardcoded notion of which formats count as data and that will be wrong on some repo. It does risk crying wolf, though. Anyone who sees the notice on every second commit because of README edits will stop reading it, which costs more than the ambiguity it was added to remove.
Which formats #43 should take on first. #43 asks for non-CSV files to be handled and names xlsx as its example. Which formats earn the conversion machinery should follow what the corpus actually contains, not which one happened to surface first.
The Measurement
For each of the 374 rows in data/message_refactoring_candidates.csv, collect the extensions of the files that commit modifies.
The extension of a modified file is visible from git show --name-status alone, with no blob read at all, and the clones go under GIT_LFS_SKIP_SMUDGE the way inspect_commit.clone already does them. So no payload comes down. It needs a clone per dataset, which is not free, but it is bounded and it is a one-off.
Deliverable
A script that produces the tally, and a decision on each of the two questions above:
Report those two conclusions here. The corpus-level distribution itself is a coverage finding for the study rather than an engineering result, so it goes in the research log, not in this thread.
Two decisions in this repo are waiting on the same unknown, and one pass over the candidate clones settles both.
The Decisions
How discriminating the "not analyzed" list should be. #59 made
inspect_commit.pyname every modified file it declined to open, so that silence stops standing in for "looked at and found unchanged". It does not discriminate: aREADME.mdlands in that list beside adata.parquet. That was deliberate, because the alternative is a hardcoded notion of which formats count as data and that will be wrong on some repo. It does risk crying wolf, though. Anyone who sees the notice on every second commit because of README edits will stop reading it, which costs more than the ambiguity it was added to remove.Which formats #43 should take on first. #43 asks for non-CSV files to be handled and names xlsx as its example. Which formats earn the conversion machinery should follow what the corpus actually contains, not which one happened to surface first.
The Measurement
For each of the 374 rows in
data/message_refactoring_candidates.csv, collect the extensions of the files that commit modifies.The extension of a modified file is visible from
git show --name-statusalone, with no blob read at all, and the clones go underGIT_LFS_SKIP_SMUDGEthe wayinspect_commit.clonealready does them. So no payload comes down. It needs a clone per dataset, which is not free, but it is bounded and it is a one-off.Deliverable
A script that produces the tally, and a decision on each of the two questions above:
Report those two conclusions here. The corpus-level distribution itself is a coverage finding for the study rather than an engineering result, so it goes in the research log, not in this thread.