Draft
Conversation
Forked Transcript in Recursion V2
…s mappings - Move verifier/vm_pvs transcript preflight seeding into pre-verifier tracegen flow - Remove VerifierModule/CommitAir path from verifier subcircuit and system AIR wiring - Remove CommitmentsBus/CommitmentsBusMessage local wiring and transcript Merkle-verify placeholder path - Implement VmPvs real row/public-value population from RecursionProof/RecursionVk - Add VmPvs witness commitment field and constrain it in VmPvsAir - Wire VmPvs public_values_bus receives by per-circuit instance mapping
- add transcript bus wiring for VmPvsAir and observe fixed/fixed_no_omc/witness commitments - enforce tidx == 0 on first valid public-values row per proof/air - keep PVS consistency bus interactions commented out in verifier and vm_pvs AIRs
- compute per-proof row count directly from child_vk circuit instances - drop chip_proofs/preflight-driven pvs iteration and tidx bases - generate tidx sequentially per proof while emitting public values
[recursion-2] refactor run_preflight & refine flows before transcript fork
- switch system preflight to fresh per-chip fork transcripts and remove trunk-clone slicing - consume VmPvs alpha/beta in system flow and keep proof-shape/tower stage ownership explicit - remove fork initial_state model and update transcript trace generation to fresh-fork initialization - centralize tower preflight record+replay wiring and rename replay_tower_proof_inner to build_tower_replay_result - compute proof_shape n_logup from tower num_layers (tower_proof.proofs.len())
- use 0-based fork_id in system fork preflight observations and stored fork logs - align ProofShape forked transcript bus receives to num_present - 1 - rename local fork variables for clarity (fork_idx -> fork_id, fork_id_0 -> fork_id) - update transcript/preflight docs to describe 0-based fork identifiers
[recursion-v2] proof shape module
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
2 participants
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.
Related to #1267