Repository navigation
Conversation
…reader Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 38 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (11)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
… record the search Screens about 2,000 Commons and Openverse results; 67 photos examined at full resolution have 19 with any label field legible, against a pre-set bar of about 40, so the reading, naming and transfer questions are not measured. Adds the screening record, a tally command, tests and a path-filtered CI job. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Note Review skipped Codemetrics is installed on this repository, but it hasn't been added to your team yet. To enable AI code reviews for this repo, go to your dashboard and add house-scanning to your team's repositories. |
- docs/00 reports the first TestFlight run on a real wall (PR #23), says rules.yaml marks every value without a public source as a placeholder, and calls Gemini the planned box detector, since no branch uses it yet. - docs/04 notes that the demo has no 1 ft wall-distance rule because the solver places the battery flush. - AGENTS.md and README point to PR #23, PR #17 and the exact docs/00 section, following the rewritten prompt-writing skill. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Closing without merging. The experiment stopped at its gate: a search found 7 usable openly licensed photos of US panels, and the test needs 40. docs/00 records that in one line. The screening code and the search record stay on the branch if the photo supply changes. |
The store folder is a9-no-tape and the repo folder is recon/a9-no-tape-scale. Widen the breadth ruff ignore so that folder is linted the same way as A1 through A8.
What changed
We asked whether one phone photo of the main panel's label could replace the installer's form. The form asks for the manufacturer, the model, the main breaker amperage, and whether the panel is a known-hazard line such as Federal Pacific Stab-Lok or Zinsco. This PR adds
experiments/panel-label/. Its pass criteria and a stop condition were committed before any result: fewer than about 40 usable openly licensed photos ends the experiment.Final answer: stopped at the gate
Of the 9, two show the same panel and one is an Indian moulded-case breaker, leaving 7 distinct US panels. The hazard lines barely appear: one Stab-Lok panel and no Zinsco. So reading accuracy, manufacturer naming and the transfer of the meter's retake thresholds were not measured. As an anecdote, not a rate: Vision read the manufacturer or line name on 6 of the 8 legible US photos, and the amperage on 4.
What was searched
Filtered to CC0, public domain, CC BY and CC BY-SA:
screened.csvlists every examined photo with its URL, license, author and per-field call.uv run python -m panel_eval.screenprints the counts above.Photos to take on field day
The experiment can run as written on the team's own photos, kept in
~/house-scanning-data/paneland never committed. The pre-set bar needs about 40 panels. For each panel, with the homeowner's permission:Shoot handheld with the phone's own camera; the threshold question degrades these photos under control, so no deliberately bad shots are needed. Include several homes built between about 1950 and 1985 to catch Stab-Lok and Zinsco panels. Frame out or cover addresses on inspection stickers, names in the circuit directory, and the meter's serial number. The README has the same list.
Verification
uv run pytest -q: 3 tests pass. They cover the tally, the license filter, and EXIF rotation with the width cap.ruff checkandruff format --checkare clean.contents: read. This PR commits no transcribed labels, so it carries no meter-style leak check; port it from experiments: meter close-up reading, tap-to-confirm and two retake checks #16 when the experiment resumes with labels.Risk
Checklist
🤖 Generated with Claude Code