docs: 4Q flux measurement operator design (square + tetra) - #32
Merged
Conversation
Flux correlator matrix F_ij,P(r;T) with probe at T/2, GEVP-contracted in postprocessing (ground + excited from the same raw data); all 6 plaquette plane classes in the source canonical frame (E/B decomposition, sum rules); probe boxes reduced by a generic stabilizer engine with |G|/|stab| weights (orders: 2Q 16, diag-2Q 8, square 16, tetra 24 — asserted as unit tests, must reproduce the hand-verified viewer wedges exactly); same-config 2Q references (on-axis + diagonal) for site-by-site binding-rearrangement maps; PGM 9804004 even-d size set; raw-triplet storage with ensemble-ratio + single vacuum subtraction in postprocessing; gated on the 24^4 device-mem census. Phased F1-F4 with parity/continuity/physics gates. Tetra flux is beyond the published PGM program — flagged as such. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 03d50c4a7f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…f re-centering Codex found the section-7 site-set-equality-with-viewer gate is impossible (probe box != viewer domain: viewer square hMax=5, tetra spherical cutoff, vs my rectangular m=3 box). Verified directly against tools/geometry_viewer.html and fixed, plus a 3-agent audit surfaced more: - BLOCKING: Delta-f was defined center-relative-per-geometry, but a 2Q pair midpoint is offset from the 4Q center (d/2 for a square side) -> equal r subtracts different physical points. Now defined in ABSOLUTE coords with an unfold + re-center-to-pair-center step and a coincidence gate (s7 f). - BLOCKING: s3 repeated the impossible site-set-equality (and self-contradicted Sweights==box-volume). Replaced with predicate/orbit checks on the box. - s7 gate 1 rewritten (adversary-hardened): selector equivalence vs viewer predicates sqSymLevel/tetSymLevel/q2SymLevel on a common G-invariant domain (diagonal-2Q has no viewer predicate -> brute-force ref); orbit-COVER partition test (not a Sweights checksum) to actually exercise |stab| weights; wall-site + parity coverage requirement. - s8 costs: were flat volume/|G| (undercount ~55-65%) and d=8 was arithmetically wrong (137). Replaced with EXACT orbit-representative counts computed by brute force: square 60/84/112/144, tetra 55/91, on-axis 2Q 50/60/70/80, diag 2Q 100/144/196/256; ~15.5k passes/tdir. - Dropped the 'doubled coordinates' convention: false viewer-provenance (viewer is floor-centered, not doubled) and moot for even-d flux. - diagonal 2Q extended to d=8 so the d=8 square B-pairing has a reference. - nits: T_d->D4h square label; T0/T is eigenvector-side not flux sink T; frame map applied at measurement. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
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.
Design spec for the 4Q flux operators, requested by Petrus 2026-07-08 ("design the quark flux measurement operators! for square and tetra") — the P4 physics of the accepted 4Q spec (#28), done as a Fable advisor pass per the model routing.
Core design:
§10 has five open questions for Petrus (size set, margin, T_flux, excited-state scope, tetra validation anchor).
🤖 Generated with Claude Code