-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
added patience for early stopping #904
Conversation
@lematt1991 please consider reviewing this |
@myleott Any update on this ? |
This would be a nice addition. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@myleott has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Check locally that everything works fine. Model is uploaded to fbaipublicfiles. I fixed a few inconsistencies in the bpe encoding along the way, e.g. related to #1306.. Pull Request resolved: fairinternal/fairseq-py#904 Reviewed By: ngoyal2707 Differential Revision: D18418345 Pulled By: louismartin fbshipit-source-id: 53acb4d021581968d70430ee9babee07d6573c17
Summary: Check locally that everything works fine. Model is uploaded to fbaipublicfiles. I fixed a few inconsistencies in the bpe encoding along the way, e.g. related to facebookresearch#1306.. Pull Request resolved: fairinternal/fairseq-py#904 Reviewed By: ngoyal2707 Differential Revision: D18418345 Pulled By: louismartin fbshipit-source-id: 53acb4d021581968d70430ee9babee07d6573c17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@myleott has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@myleott has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Check locally that everything works fine. Model is uploaded to fbaipublicfiles. I fixed a few inconsistencies in the bpe encoding along the way, e.g. related to facebookresearch#1306.. Pull Request resolved: fairinternal/fairseq-py#904 Reviewed By: ngoyal2707 Differential Revision: D18418345 Pulled By: louismartin fbshipit-source-id: 53acb4d021581968d70430ee9babee07d6573c17
Summary: Pull Request resolved: fairinternal/fairseq-py#959 Added patience flag to support early stop when the validation metric does not improve for certain no of patience epochs Pull Request resolved: facebookresearch#904 Differential Revision: D19165379 Pulled By: myleott fbshipit-source-id: 126d779ff6b10b48bae42213329baa296bf5ca50
Summary: Pull Request resolved: fairinternal/fairseq-py#959 Added patience flag to support early stop when the validation metric does not improve for certain no of patience epochs Pull Request resolved: facebookresearch/fairseq#904 Differential Revision: D19165379 Pulled By: myleott fbshipit-source-id: 126d779ff6b10b48bae42213329baa296bf5ca50
Summary: Pull Request resolved: fairinternal/fairseq-py#959 Added patience flag to support early stop when the validation metric does not improve for certain no of patience epochs Pull Request resolved: facebookresearch/fairseq#904 Differential Revision: D19165379 Pulled By: myleott fbshipit-source-id: 126d779ff6b10b48bae42213329baa296bf5ca50
Added patience flag to support early stop when the validation metric does not improve for certain no of patience epochs