Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Earlystopping patch #1487

Merged
merged 3 commits into from Jul 31, 2023
Merged

Conversation

scumatteo
Copy link
Contributor

Solved problem by moving the reset in before_training_exp, to reload always the model adapted at the current experience.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5716014779

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.001%) to 72.971%

Totals Coverage Status
Change from base Build 5693163363: -0.001%
Covered Lines: 16857
Relevant Lines: 23101

💛 - Coveralls

@AntonioCarta
Copy link
Collaborator

Thank you

@AntonioCarta AntonioCarta merged commit 8949fbd into ContinualAI:master Jul 31, 2023
18 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.

None yet

3 participants