Skip to content
This repository was archived by the owner on Sep 8, 2025. It is now read-only.

remove hardcoded n_epochs#23

Open
johnlyzhou wants to merge 1 commit into
themattinthehatt:masterfrom
johnlyzhou:patch-1
Open

remove hardcoded n_epochs#23
johnlyzhou wants to merge 1 commit into
themattinthehatt:masterfrom
johnlyzhou:patch-1

Conversation

@johnlyzhou

Copy link
Copy Markdown
Contributor

remove the hardcoded conditional requiring n_epochs==200

remove the hardcoded conditional requiring n_epochs==200
@themattinthehatt

Copy link
Copy Markdown
Owner

@johnlyzhou good catch here. We'll still need to add some additional updates though - filtering by n_epochs ensures that we only plot the loss values at the end of training; not doing so will plot the loss values for all epochs, which we don't want. So you'll have to find the max number of epochs in the dataframe, then filter by that value.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants