Repository navigation
K7: scope variants, context phases, one execution result family - #1209
Merged
Merged
Conversation
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>
…se split 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>
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>
Result subclasses replace E1's ExecutionStatus family; E1's observation-mode family is kept. NamedSourceBinding.input_plan deleted (unused). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… follow-up, caches) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…re progress fixture import Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Materialization annotates ExecutingContext; execution_axis_count lives on it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
trissim
added a commit
that referenced
this pull request
Oct 12, 2026
The installed-GUI smoke keeps formbridge's bridge DTO paths and the datasets widget identity, and reads the action status as the UiActionInvocationStatus class family from formbridge.bridge.dto. 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.
K7 · PR #1209 · guards 10/10 · production −1331 +1248 · tests −1462 +1684
Surface K7 of the consolidation refactor. Receipt:
docs/refactor/consolidation-20261009/K7-execution-protocol.md.What changed
UngroupedScope,DynamicGroupScope,StaticGroupScope,SingleKeyScopereplace the(keys, component)packing and its six predicates (48 consumer reads, now 0).ArtifactPlanholds onegroup_scopefield instead ofgroup_keys+group_component.PathPlannerGroupScopeis deleted.RuntimeValue.compose(values)andcompose_groups(values, producer_scope)replace theproducer_group_scope=Nonemode.CompilingContext→freeze()→ immutableCompiledContext→executing(lane)→ExecutingContextwith that execution's stores and caches._is_frozenand its seven checks are deleted. The parent's executed contexts come back onBundleExecutionResults.executed_contexts.ExecutionSucceeded,ExecutionFailed,ExecutionCancelledreplaceExecutionStatusplus optional fields;RuntimeObservationModeis a family (E1's member names).Behaviour to know about
bundle.runtime_contexts[k].runtime_value_storeafter execution readsresults.executed_contexts[k]instead.ExecutionFailed: …where it readerror: ….Deletion certificates and the tests-only list are in the receipt.
Overlap
ExecutionStatusandRuntimeObservationModeinexecution_result.py. This PR uses E1's mode names; the result subclasses replace E1's status family. Whichever merges second resolves the conflict.NamedSourceBinding.input_plan(dead, names the deleted plan fields; E1's file) and theProcessingContextannotations inprocessing/materialization/(K8).Checks
Run locally at
f528cb67fagainst based597aa224:tests/unit/test_k7_execution_protocol_guards.py: 10 passed. The other 23 guard files: 195 passed, 1 failed (test_cellprofiler_static_deletion_gates.py::test_cellprofiler_projection_enters_only_runtime_slice_projection, which also fails on the base).-n 2): 39 failed, 4616 passed; the base has 40 failed on the same files. Three failures were not on the base: two intest_zmq_runtime_outcome_export.py(fixed afterwards, file now passes) andtest_axis_family_witness.py[zmq], whose client reached another worktree's execution server; it passes when run alone.tests/integration/test_main.pydisk, three microscopes, direct and ZMQ, with the witness: 19 passed, 1 skipped.test_official30_compile_execute_and_match_native_references_over_zmqpassed (979 s).openhcs,scripts,benchmark): exit 0. nra-r1:"increased": [].The full unit suite was not run (the session lead verifies centrally).
🤖 Generated with Claude Code