Skip to content

feat: learn every N steps with configurable frequency#9

Merged
jdbloom merged 1 commit intomasterfrom
feat/learn-every-n-steps
Apr 10, 2026
Merged

feat: learn every N steps with configurable frequency#9
jdbloom merged 1 commit intomasterfrom
feat/learn-every-n-steps

Conversation

@jdbloom
Copy link
Copy Markdown
Collaborator

@jdbloom jdbloom commented Apr 10, 2026

Summary

  • Adds LEARN_EVERY config parameter (default: 4)
  • Increases BATCH_SIZE from 64 to 256
  • Increases GSP_BATCH_SIZE from 128 to 256
  • Same 256 samples per 4 steps, 75% fewer gradient updates

Test plan

  • All existing tests pass
  • Characterization run comparing timing before/after

🤖 Generated with Claude Code

Learn every 4 steps with batch_size=256 maintains same sample
throughput as every-step with batch_size=64 (256 samples/4 steps)
while reducing gradient update overhead by 75%.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jdbloom jdbloom force-pushed the feat/learn-every-n-steps branch from dd08bc4 to c58c948 Compare April 10, 2026 13:17
@jdbloom jdbloom merged commit 6aed339 into master Apr 10, 2026
3 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.

1 participant