chore(bb): refresh pinned Chonk IVC inputs (fixes nightly debug build)#23956
Closed
AztecBot wants to merge 1 commit into
Closed
chore(bb): refresh pinned Chonk IVC inputs (fixes nightly debug build)#23956AztecBot wants to merge 1 commit into
AztecBot wants to merge 1 commit into
Conversation
40dd610 to
5f4588b
Compare
Collaborator
Author
|
Pinned Chonk inputs refreshed to |
Fixes the Nightly Debug Build failure in bbapi_tests ChonkPinnedIvcInputsTest.AllPinnedFlows: the pinned Chonk IVC inputs had drifted from the current protocol circuits (hiding_kernel public-input layout 669 -> 673), tripping the debug-only databus consistency assert in chonk.cpp. Inputs regenerated via the ci-refresh-chonk path; this is the freshly-captured hash. --ci-skip
364cfa0 to
1ce7d66
Compare
Collaborator
Author
|
Automatically closing this stale claudebox draft PR (no updates for 5+ days). Re-open if still needed. |
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.
What
Refreshes the pinned Chonk IVC inputs to
49442f24d1be75a0(barretenberg/cpp/scripts/chonk-inputs.hash). The new tarball was regenerated and uploaded by theci-refresh-chonkpost-action from the current protocol circuits, and self-verified by proving the smallest refreshed flow in C++ and bb.js (which re-runs the exact failing test). Single commit, hash-only change.Why — Nightly Debug Build failure
Nightly Debug Build failed every night for ≥4 days (runs #109–#112, Jun 6–9), latest on
nextHEADd9b6e71:bbapi_tests ChonkPinnedIvcInputsTest.AllPinnedFlowsRoot cause — stale pinned inputs
Accumulating circuit 10/10 (
hiding_kernel), the old pinned VK (209dde8e69a27c9f) no longer matched the circuit built from current code:The kernel public-input layout grew by 4, so
KernelIO::reconstruct_from_publicread misaligned fields and the databus consistency check inchonk.cpp(process_public_inputs_and_consistency_checks) tripped:{0x..01, 0x..02}is the BN254 generator (an empty databus column) vs real return data in the proof — the "pinned inputs drifted from the protocol circuits" signal. It surfaces only in the debug nightly because the check isBB_ASSERT_DEBUG(compiled out of release;chonk_inputs.sh checkruns--disable_asserts), which is why merge-queue CI stayed green while breaking protocol-circuit PRs merged withoutci-refresh-chonk. No bb product code is changed here.Notes
--ci-skip(hash-only refresh, already self-verified by the refresh post-action), matching the standard refresh-PR convention.draft == false || ci-draft); this PR usedci-draftso the refresh executed.Ready to merge to
nextonce marked ready-for-review.