Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[train] Fix bugs with loading validation impatience. #3713

Merged
merged 3 commits into from
Jun 15, 2021

Conversation

stephenroller
Copy link
Contributor

Patch description
Validation impatience has been loading incorrectly upon preemption for a bit.

The following two bugs are observed and fixed:

  • .checkpoint was saved BEFORE the validation report had happened, meaning it would always save epochs/progress/etc with the latest values, but the "best validation score" of the PREVIOUS validation.
  • The impatience key was not being set, and therefore always being loaded with 0 (the default).

Testing steps
About to write unit tests.

@stephenroller stephenroller changed the title [train] Fix bug with loading validation impatience. [train] Fix bugs with loading validation impatience. Jun 14, 2021
@jaseweston
Copy link
Contributor

awesome thanks!

@mojtaba-komeili
Copy link
Contributor

Thank you for quick fix!

@stephenroller stephenroller marked this pull request as ready for review June 15, 2021 12:56
@stephenroller stephenroller merged commit 02d4ae7 into master Jun 15, 2021
@stephenroller stephenroller deleted the valimpatience branch June 15, 2021 12:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants