Skip to content

count completed training steps - #58

Open
borawhocodess wants to merge 1 commit into
automl:abstractionfrom
borawhocodess:validsteps
Open

borawhocodess wants to merge 1 commit into
automl:abstractionfrom
borawhocodess:validsteps

Conversation

@borawhocodess

Copy link
Copy Markdown
Collaborator

follows the remaining continue statement comment in #40

steps_per_epoch now counts optimizer updates, not batches drawn

in this pr:

  • counts optimizer updates toward steps_per_epoch
  • retries skipped batches
  • raises after 100 consecutive skips

not here:

  • turning epochs into steps future work #44
  • closing the sources of non-finite batches (removing continue all together)

Co-Authored-By: Codex <noreply@openai.com>
@borawhocodess borawhocodess self-assigned this Sep 12, 2026
@borawhocodess borawhocodess linked an issue Sep 12, 2026 that may be closed by this pull request
@borawhocodess
borawhocodess changed the base branch from main to abstraction September 12, 2026 08:06
This was referenced Sep 12, 2026
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.

unify it all

1 participant