Skip to content

specs: layernorm, wrapup-auto and ternary_mac_synth test what they claim (Closes #7404) - #7418

Merged
gHashTag merged 8 commits into
shep-tmp-base-7418from
t27b-lane5-spec-defects
Oct 8, 2026
Merged

gHashTag merged 8 commits into
shep-tmp-base-7418from
t27b-lane5-spec-defects

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Closes #7404
Part of #6063

Three specs were blocked in the t27b ledger because of defects in the specs themselves, not in t27b.

  • specs/ml/layers/layernorm_layer.t27: forward assigned to names it never declared, so no backend could run it. It now computes the mean and the biased variance and writes (x - mean) / sqrt(variance + 1e-5). Five tests check known outputs, an empty input, a single element and a constant input. The reference passes 5/5 with 0 vacuous tests, and each of 8 hand mutants of forward fails at least one test.
  • specs/automation/wrapup-auto.t27: the only test had no braces, so the reference ran nothing. wrapup_run is Python and is marked NOT CHECKED. The test now builds a WrapUpInput and checks its fields at run time (5 runtime asserts).
  • specs/port/fpga/verilog/ternary_mac_synth.t27: the reference's Zig backend printed [a,b]u8 as .{ _ }, so 13 uses now read [a, b]. gen-verilog, gen-c and gen-rust output is byte-identical. The reference passes 10/10, with 0 vacuous.

Seals were regenerated on the t27c lab, and check_seal_coverage and check_seal_currency both exit 0 there.

t27b ledger, from a targeted corpus run on the t27b lab:

spec before after
layernorm_layer blocked StmtAssign(undeclared) pass (17 asserts)
ternary_mac_synth blocked ExprArrayLiteral(reference empty typed) pass (11 asserts)
wrapup-auto blocked StmtAssign blocked ExprArrayLiteral(to slice field)

max_not_pass goes from 24 to 22 (recounted after the latest master merge 26ebb95: 822 entries, no duplicate paths). The ledger is master's ledger with only these three rows changed.

No Rust is touched.

🤖 Generated with Claude Code

gHashTag and others added 3 commits October 7, 2026 15:56
…aim; reseal; ledger (Closes #7404)

layernorm_layer and ternary_mac_synth move blocked -> pass in the t27b
ledger; wrapup-auto stays blocked on ExprArrayLiteral(to slice field).
max_not_pass 25 -> 23.

Part of #6063

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

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-07 10:11:12 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 38
PRs with All Checks Green 12
READY 1
FAILING 38
PENDING 0
NO CHECKS YET 0

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

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=1aa228450491 != 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).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 8, 2026
- .trinity/seals: the five seal files of layernorm_layer, wrapup-auto and
  ternary_mac_synth are regenerated with `t27c seal --save`, t27c built
  from 927b1ef on the t27c lab (the merged tree has master's
  bootstrap/); `--verify` gives all hashes MATCH. Gen hashes are
  unchanged; only built_by and sealed_at move.
- Ledger: master's, with only this PR's three rows: layernorm_layer and
  ternary_mac_synth blocked -> pass, wrapup-auto's blocker StmtAssign ->
  ExprArrayLiteral(to slice field) (not_pass 51 -> 49).

Closes #7404

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

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-08 07:29:32 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 44
PRs with All Checks Green 6
READY 2
FAILING 44
PENDING 0
NO CHECKS YET 1

These columns do not partition: 2 + 44 + 0 + 1 = 47, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=39fa3908f9fd != 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).

Master moved (#7414 landed), and the ledger conflicted again.

- Ledger: master's (12c41a4), with only this PR's three rows, the same
  rows a bless of the lab run of b69c63c gives: layernorm_layer and
  ternary_mac_synth blocked -> pass, wrapup-auto's blocker StmtAssign ->
  ExprArrayLiteral(to slice field) (not_pass 50 -> 48).
- .trinity/seals: the five seal files resealed with `t27c seal --save`,
  t27c built from 12c41a4 on the t27c lab; `--verify` all MATCH. Gen
  hashes unchanged.

Closes #7404

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

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-08 08:55:11 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 45
PRs with All Checks Green 5
READY 4
FAILING 45
PENDING 0
NO CHECKS YET 0

These columns do not partition: 4 + 45 + 0 + 0 = 49, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=39fa3908f9fd != 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).

@gHashTag
gHashTag changed the base branch from master to shep-tmp-base-7418 October 8, 2026 08:56
@gHashTag
gHashTag merged commit 4317dd2 into shep-tmp-base-7418 Oct 8, 2026
30 of 32 checks passed
@gHashTag

gHashTag commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

Merge shepherd: auto-merge squashed this PR into the temporary branch shep-tmp-base-7418 while I was flipping its base to clear a stale base.sha (#7489). That branch is deleted, and nothing reached master. The same head, 281d5b4, continues as #7745. The receipt verdict for this content is IMPROVED_ONLY (exit 3), posted there.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

t27b lane 5: three specs whose tests do not test (layernorm, wrapup-auto, ternary_mac_synth)

2 participants