Add option to print loss every printInterval epochs in PrintLoss callback.
#442
Conversation
There was a problem hiding this comment.
The implementation looks correct enough to me but I have to ask, is there really any need for this? PrintLoss is meant to be a really primitive callback. I haven't ever seen anyone ask for this particular feature and I'm not actually sure the situation where someone would want to use it.
| const double objective) | ||
| { | ||
| output << objective << std::endl; | ||
| // epochs are 1-indexed. |
There was a problem hiding this comment.
Are they? I thought they started at 0. In either case, it doesn't really make a big difference: if they are 0-indexed they will print the first iteration too.
|
I figured it might help, since with lots of iterations the terminal can get flooded and we lose all previous output. If we don't print anything, though, there's no sense of progress. I also wanted to include the epoch number in the output, but didn't want to change the current behavior. Happy to leave it as is if you think keeping |
2 participants
Description
A small addition to the
PrintLosscallback which allows to print loss value everyprintIntervalepochs. The change is backwards compatible (printIntervalhas a default value of 1).