Skip to content

Conversation

@vijk777
Copy link
Collaborator

@vijk777 vijk777 commented Jan 20, 2026

  • remove train step evaluation on validation data, it is not needed
  • deprecate ems_warmup
  • fix validation

vijk777 and others added 3 commits January 20, 2026 14:55
validation metrics are now only computed via diagnostics (rollout stats).
this removes the redundant per-epoch validation train_step call.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
the ems warmup feature was not being used and added complexity.
removed the config field, validation logic, and training loop handling.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…ochs

warmup epochs run before the main training loop and are additional to
the configured 'epochs' count. removed misleading validation that
required warmup < epochs.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vijk777 vijk777 merged commit 0e22d64 into main Jan 21, 2026
2 checks passed
@vijk777 vijk777 deleted the vj/stream-batches branch January 21, 2026 01:48
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.

2 participants