Skip to content

Always store best_score - #439

Merged
nreimers merged 1 commit into
huggingface:masterfrom
PhilipMay:always_store_best_score
Oct 2, 2020
Merged

Always store best_score#439
nreimers merged 1 commit into
huggingface:masterfrom
PhilipMay:always_store_best_score

Conversation

@PhilipMay

Copy link
Copy Markdown
Contributor

This is just a small fix.

  • the best_score during training had been stored only when save_best_model was true
  • now best_score is always stored

- the best_score during training had been stored only when save_best_model was true
- now best_score is always stored
@PhilipMay

Copy link
Copy Markdown
Contributor Author

Just wanted to bump this. @nreimers

@nreimers
nreimers merged commit dc84bb7 into huggingface:master Oct 2, 2020
@nreimers

nreimers commented Oct 2, 2020

Copy link
Copy Markdown
Collaborator

Sorry for the delay. Thanks for the PR, it is merged :)

@PhilipMay

Copy link
Copy Markdown
Contributor Author

Yay! ;-)

@ReySadeghi

Copy link
Copy Markdown

Hi,
save best model by default is based on cosine_average_precision?

@nreimers

Copy link
Copy Markdown
Collaborator

@ReySadeghi
It is based on what the evaluator, passed to the fit method, returns.

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.

3 participants