Repository navigation
inference-summary@3 record shards; fit pinned @3 (migrating_from @1); gaussian_x3 wave 2 closed (B5b-2) - #31
Merged
Conversation
…ing_from @1 (B5b-2) Reader: v3 index (record_shards: id, protocol_id, path, sha256, records) with shard files verified by sha256, header and count, assembled and validated as v1. Ingest keeps shard bytes under snapshots/<instance>/, re-fetches only changed shards at the resolved commit, writes them only after the capture validates and removes orphans; offline re-verifies; --from reads shards beside the index. Board lists shards, not every record. Registry migrating_from keeps the committed @1 fit capture valid until the first @3 refresh. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…; board --offline Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rating_from note, v3 parity disclaimer (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ad wording (B5b-2) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Jammy2211
pushed a commit
that referenced
this pull request
Oct 15, 2026
The committed fit capture is inference-summary@3 since the post-#31 refresh; the acceptance test still indexed doc['records'] and failed lint on this branch (and latently on main, whose refresh commit skipped CI). Co-Authored-By: Claude Fable 5.1 <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 the PyAutoInsight part of B5b-2 (search-extensibility epic; autofit_inference#12, producer PR PyAutoLabs/autofit_inference#13). It adds the
inference-summary@3contract:@1's envelope and record contract, with the records held in sha256-verified shard files. It also moves thefitinstance to@3.What changes:
insight/summary.py).@3index carriesrecord_shards(id,protocol_id,pathrelative to the index,sha256,records) and never inline records.load_shardedchecks each shard's digest, header (inference-summary-records, v3, project, shard id, protocol) and record count, and checks that each record's protocol matches its shard's.@1.@1and@2are unchanged.insight/ingest.py).snapshots/<instance>/. They are written only after the whole capture validates, then the index is written, then shards the new index no longer names are pruned.--fromreads the shards beside the index.fitis pinned tosupported_schema: inference-summary@3, withmigrating_from: inference-summary@1. Until the first@3capture, the committed@1snapshot stays valid last-good evidence, labelled "awaiting the first refresh". A fresh fetch must still return@3.checksays whenmigrating_fromhas become obsolete.REFERENCE.mdgains a v3 section.tests/test_sharded.pyadds 18 tests.tasks/gaussian_x3_search_wave2.mdand the campaign next/progress lines are updated (wave 2 closed). The board is re-rendered--offline.Not done here, by design: the online
fitsnapshot refresh. It needs autofit_inference#13 onmain. When this PR merges after it, the push-triggeredDashboard Refreshcapturesfitunder@3. Then runbin/pyauto-insight checkonline and dropmigrating_from.unsupportedreceipt forfitin between.Dashboard Refreshdry run reads autofit_inferencemain. Until feat: add inference setup and baseline experiment contract #13 merges,fitshows asunsupportedin that dry run (cached@1evidence is still shown).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.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 autofit_inference#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