Skip to content

specs/numeric: GF8/20/24/32 invariants check concrete points instead of prose the parser drops - #5699

Closed
dmitrii-f-t27 wants to merge 1 commit into
masterfrom
fix/gf-prose-invariants
Closed

dmitrii-f-t27 wants to merge 1 commit into
masterfrom
fix/gf-prose-invariants

Conversation

@dmitrii-f-t27

Copy link
Copy Markdown
Collaborator

Closes #5698

t27c parse-complete dropped 37 tokens from gf8 and 31 each from gf20, gf24 and gf32. These were prose invariants (for all positive x, for all valid x, floor(x) == i32 for all f32 x, and in gf8 PHI_DISTANCE == 0.132 within 0.001). Each invariant kept its name and lost its body, so nothing was checked.

  • Each claim is now checked at concrete points: 0.5, 2.5 and 7.0 for pow; 3.7 and -3.2 for floor via floor(floor(x)) == floor(x). The gf8 phi distance uses abs(...) < 0.001.
  • parse-complete --show now prints nothing discarded for all four specs.
  • Typecheck is unchanged (OK, 12 warnings).
  • Zig ast-check gives 3 errors on master and 3 on this branch: the same pre-existing unused capture, which also blocks test-report on master.
  • The 8 seals were regenerated with the t27c built from this tree. --verify reports all hashes MATCH.

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-03 09:56:42 UTC

Summary

Status Count
Total Open PRs 36
PRs with Failing Checks 34
PRs with All Checks Green 2
READY 1
FAILING 34
PENDING 0
NO CHECKS YET 0

These columns do not partition: 1 + 34 + 0 + 0 = 35, and there are 36 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=b45a356c2eb6 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@dmitrii-f-t27

Copy link
Copy Markdown
Collaborator Author

Superseded by #5702: same change, with "Closes #5698" in the commit message (L1 traceability) and the four fixed specs removed from the expected-failure ledger (corpus ratchet).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GF8/20/24/32 specs: prose invariants are discarded by the parser and check nothing

1 participant