Closed Bartolo1024 closed 4 years ago
1) regex looks better a lot, first value as default is less clear.
2) I unnecessary added it to all and not reverted this one.
3) Can you write something more? I did not understand. Currently, we have "Validation Accuracy"?
4) It is done since commit 3, I did not change it in comment.
5) ok
Ad 3 and 4 - the result should be as in:
After this thing, I think we can merge, release, and move on. :)
I did it, but I have to get metrics_to_name from logger in ExtremaPrint and do transformation there, because it requires unique names. Optionally I can do cache for each group in ExtremaPrint. Which approach seems to be better?
I looked at the code, and while there is some progress, it is not what I had in my mind.
[('', 'Training'), ('^val_', 'Validation')]
. Or in the 'Training' actually need negative look before: https://stackoverflow.com/questions/899422/regular-expression-for-a-string-that-does-not-start-with-a-sequence (but maybe not, just leaving the first option as default). Note that thanks to the pattern we can easily use re.replacelog_history
got an underscore. It shouldn't.Negative Log-Likelihood
. But in out case, we actually wantLog Loss
.Look at the previous outputs, it should look similarish.