Skip to content

feat: GF-T trainer size optimizations (scale_q + signmul) - #1839

Merged
gHashTag merged 1 commit into
masterfrom
feat/gft-xorpercep3
Aug 7, 2026
Merged

feat: GF-T trainer size optimizations (scale_q + signmul)#1839
gHashTag merged 1 commit into
masterfrom
feat/gft-xorpercep3

Conversation

@gHashTag

@gHashTag gHashTag commented Aug 7, 2026

Copy link
Copy Markdown
Owner

New spec gft_xorpercep3.t27 — fully-on-chip 2-layer XOR trainer with two reusable, numerically-identical optimizations (in-spec PASS): scale_q(x,k)=x*2^-k via exponent shift (replaces smul(eta,.) for power-of-2 eta); signmul(g,h)=sign/zero mux (valid since perceptron error g is exactly {-1,0,+1}, replaces smul(g,.)). Shrinks 19.5M->17.86M fasm (6 magmuls->2). Honest note: still above the ~17M correctness ceiling — the bulk is magsub (normalize in every sadd), not magmuls; a full 2-layer train step is irreducibly ~17.9M. Working on-chip path stays the split. docs/NOW.md updated. Refs #1764

gft_xorpercep3.t27: fully-on-chip 2-layer XOR trainer with two reusable,
numerically-identical size optimizations (in-spec test PASS):
- scale_q(x,k) = x*2^-k via exponent-offset shift, replaces smul(eta,.) for a
  power-of-2 eta (no multiplier)
- signmul(g,h) = sign/zero mux, valid because the perceptron error g is exactly
  {-1,0,+1}, replaces smul(g,.)

Shrinks the design 19.5M -> 17.86M fasm (6 magmuls -> 2). Honest note: still
above the ~17M correctness ceiling -- the bulk is magsub (normalize loop in
every sadd), not the magmuls, so multiplier optimizations do not clear the
ceiling; a full 2-layer train step is irreducibly ~17.9M. Working on-chip path
stays the split. scale_q/signmul are reusable for any GF-T trainer near budget.

Refs #1764

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

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-08-07 06:55:43 UTC

Summary

Status Count
Total Open PRs 4
PRs with Failing Checks 0
PRs with All Checks Green 4
READY 3
FAILING 0
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=4c1aaad5309a != 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 Aug 7, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

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

@gHashTag
gHashTag merged commit 4e371f7 into master Aug 7, 2026
17 checks passed
@gHashTag
gHashTag deleted the feat/gft-xorpercep3 branch August 7, 2026 06:56
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.

1 participant