Skip to content

specs/ml: race_config literals lose the '...' the parser dropped; digits unchanged - #5741

Merged
dmitrii-f-t27 merged 1 commit into
masterfrom
fix/race-config-elided-literals
Oct 3, 2026
Merged

dmitrii-f-t27 merged 1 commit into
masterfrom
fix/race-config-elided-literals

Conversation

@dmitrii-f-t27

Copy link
Copy Markdown
Collaborator

Closes #5740

specs/ml/optimizer/race_config.t27 wrote two f64 constants with an elided tail (0.5559321608..., 0.6174052591...). The parser dropped the ... (4 tokens) and kept the digits.

  • The ... is removed and the digits are kept, so t27c gen output is byte-identical to master's.
  • parse-complete --show: 4 token(s) DISCARDED -> nothing discarded. Typecheck unchanged (OK, 0 warnings); ast-check 1 pre-existing error before and after (elapsed_time_ns), which also blocks test-report on both.
  • Not decided here: the digits differ from the quotients their comments name (0.9/phi = 0.5562305899, 0.999/phi = 0.6174159548; adamw.t27 uses 0.9 / PHI). A comment records the mismatch for the owner.
  • Seal optimizer_RaceConfig.json regenerated with the t27c built from this tree; --verify reports all hashes MATCH.
  • The parse-no-discard entry is removed from docs/reports/suite_expectations.json; local t27c suite --ratchet --corpus-only: RATCHET CLEAN.

🤖 Generated with Claude Code

…its unchanged (#5740)

RACE_BETA1/2_PHI_DAMPED were written '0.5559321608...' and '0.6174052591...'.
The parser dropped the ellipsis and kept the digits, so the generated code
is byte-identical after this change. A comment records that the digits
differ from 0.9/phi and 0.999/phi in the fourth decimal and leaves the
choice to the owner. Seal regenerated with this tree's t27c; the
parse-no-discard ledger entry is removed.

Closes #5740

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-03 10:28:32 UTC

Summary

Status Count
Total Open PRs 45
PRs with Failing Checks 42
PRs with All Checks Green 3
READY 1
FAILING 42
PENDING 0
NO CHECKS YET 0

These columns do not partition: 1 + 42 + 0 + 0 = 43, and there are 45 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 2f5d03f into master Oct 3, 2026
34 of 36 checks passed
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.

race_config: two f64 literals end in '...', which the parser drops (and their digits differ from 0.9/phi)

1 participant