Repository navigation
B5b-2: wave-2 remainder judged, inference-summary@3 record shards, catalogue reliability floor - #13
Merged
Merged
Conversation
… accounting (B5b-2) Pulled results/ only by a 12-way per-directory tar on the RAL login node; the 20 probe rows came back byte-identical. 12,750 wave-2 rows admitted, 0 refused, 0 missing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ity floor (B5b-2) Exporter writes the @3 index (dashboard/summary.json, record_shards) and one shard per protocol x leg under dashboard/records/ (ruling 1); the fixture test validates natively when the Insight checkout reads @3 and as assembled @1 records otherwise. The catalogue ranks only legs with success rate >= 50 % over >= 20 attempts and lists the rest as unreliable (ruling 2, post-hoc presentation convention); README rows mark them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…and wave-2 close (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…loor (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… 101 record shards (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…unes emptied folders (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…-2 review) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Summary
This is B5b-2 of the search-extensibility epic. It closes scored wave 2: it pulls and judges the probe-first remainder (RAL job 409930), publishes the wave-2 records under a sharded summary contract, and applies the catalogue-ordering ruling. Closes #12.
Remainder.
COMPLETED, from 16:08 to 17:38 UTC (wall 1:30).judge_wave2.pyadmits all 12,750 wave-2 rows at the pindf6d218f10cd, with 0 refused.The probe-first legs under
gaussian_x3@2:No run came near its timeout: NSS took at most 29 s against a 600 s timeout, and Zeus at most 131 s against 1168 s.
Ruling 2 (ordering).
unreliablewith their numbers. In the README they show "unreliable" in the # column.RANK_FLOORlives inbuild_catalogue.py.Ruling 1 (records).
dashboard/summary.jsonis now aninference-summary@3index. Itsrecord_shardslist names 101 per-protocol, per-leg files underdashboard/records/, each with its sha256 and record count, and each written one record per line.@1verdicts in thegaussian_x3@1shards.@3. Otherwise it validates them as assembled@1records, which is the case in CI today, against Insight main.Commits.
96008ba.7c7af64.b9f5927.c3a81d1.74b834f.72845c0,6a22a95,719c2f3,ed38ade,035d4f4.output/is never committed.Decisions taken (decide-and-flag)
inference-summary@3, not@2. This needs your sign-off. Ruling 1 named@2, but PyAutoInsight's@2already exists: it is the setups/prepared-problems contract that instancelensis pinned to. Reusing the number would have brokenlens. The per-protocol split therefore takes the next free version,@3. It keeps@1's envelope and record contract unchanged and holds the records in shard files.@3does not include@2's setups.gaussian_x3@2file would hold 12,750 records and still be far above 50 MB. The ruling allowed "an equivalent per-leg layout", so there is one shard per protocol × dataset × search × settings × config. That gives 101 shards; the largest is 1.56 MB and the total is 66 MB. Each shard is written one record per line, and each shard's sha256 is in the index, so a refresh re-copies only the shards that changed.@3withmigrating_from: inference-summary@1. Without that field, the committed@1fitsnapshot would failcheck --offlinefrom the moment the pin moves. With it, the old capture stays valid last-good evidence, labelled "awaiting the first refresh". A fresh fetch must still return@3.Order and next steps (human)
Merge in this order: this PR → PyAutoCortex → PyAutoInsight.
@1reader. If it runs between the two merges, it records anunsupportedreceipt forfitand Insight'scheck --offlinegoes red until the next refresh.Dashboard Refreshdry run reads this repository'smain, so it can only go green after this PR merges.Dashboard Refreshcapturesfitunder@3. Then runbin/pyauto-insight checkonline and dropmigrating_fromfrom Insight'sregistry.yaml. Ruling 1's "check green online" is therefore not met yet: it is satisfied only by that post-merge capture.Ship gate
--checkgates green. PyAutoInsight: 204 passed; ruff clean;check --offlineOK;board --offlinerendered. PyAutoCortex: 65 passed;cortex.py checkOK.check --from fit=<this branch>validates all 12,973 records in 101 shards.@2at df6d218f1, and the four probe-sourced legs closed: three positioned, Zeus JAX blend recorded at 0/250, unreliable. ✓ Ordering ruled and protocol amended as a post-hoc §8 note (plus N1 and astate.mdruling line). ✓ Cortex 409930 moved into the log with wall 1:30. ◐ Records reach Insight under the chosen schema: shipped as@3(see decision 1), with the registry bumped andcheck --offlinegreen;checkonline and thefitsnapshot refresh frommainwait for the merges (human next step).Heart RED — development-only override (quoted from #12)
Heart at the door:
Heart: RED — Release RED; monitoring RED · 10/100 · 415 unresolved;[red] release validation FAILED (stage integrate);[red] PyAutoHands/workflows/Python Version Matrix: failure;[red] PyAutoHeart/workflows/Release Integrate: failure;[red] PyAutoMind/workflows/Lifecycle Drift: failure(cleared 2026-10-15 evening);[red] PyAutoArray/Tests: red;[red] autofit_workspace_test/Smoke Tests: red;[red] autogalaxy_workspace_test/Smoke Tests: red;[red] autolens_workspace_test/Smoke Tests: red;[red] Release validation: NOT release_ready;[red] Worktree drift;[red] autolens_inference: red;[red] autolens_profiling: red;[red] euclid_strong_lens_modeling_pipeline: red. Live human in session 6ffb59e9 answered AskUserQuestion "Yes, override for development (Recommended)" on 2026-10-15 for this launch. Permitted: commit, push, PR-open. Merge is a separate human act; this never reaches a release.🤖 Generated with Claude Code