Skip to content

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

Merged
dmitrii-f-t27 merged 1 commit into
masterfrom
fix/gf-prose-invariants-v2
Oct 3, 2026
Merged

dmitrii-f-t27 merged 1 commit into
masterfrom
fix/gf-prose-invariants-v2

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 4 parse-no-discard entries are removed from docs/reports/suite_expectations.json (max_entries 134 -> 130), as the corpus ratchet asks.
  • The 8 seals were regenerated with the t27c built from this tree. --verify reports all hashes MATCH.

🤖 Generated with Claude Code

…of prose the parser drops

The parser discarded three prose invariants in each spec (37 tokens in gf8,
31 in gf20/gf24/gf32); their names stayed and their bodies were gone. Each claim
is now checked at concrete points, the eight seals are regenerated with the
t27c from this tree, and the four parse-no-discard entries leave the
expected-failure ledger (max_entries 134 -> 130).

Closes #5698
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-03 10:01:53 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).

@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.

@dmitrii-f-t27
dmitrii-f-t27 merged commit 146321c into master Oct 3, 2026
35 of 37 checks passed
dmitrii-f-t27 added a commit that referenced this pull request Oct 3, 2026
Resolves the ledger conflict with #5702: keep master's ledger and drop only
the gf12 parse-no-discard entry.

Closes #5723
dmitrii-f-t27 added a commit that referenced this pull request Oct 3, 2026
…0 -> 126

#5702 also lowered the cap from 134 to 130, so the two edits merged
silently to 130. This branch removes four more entries, so the cap is 126.

Closes #5728

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
dmitrii-f-t27 added a commit that referenced this pull request Oct 3, 2026
Closes #5728

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
dmitrii-f-t27 added a commit that referenced this pull request Oct 3, 2026
…follows #5702 down

Closes #5735

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
dmitrii-f-t27 added a commit that referenced this pull request Oct 3, 2026
Closes #5735

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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