Details
-
Bug
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
Description
In /src/utils/common.cc, inside the loop of "void Metric::Reset()" function, the reference of entry needs to be reset instead of the variable.
Previous:
for(auto e : entry_)
Fix the bug:
for(auto &e : entry_)
If we use the previous reset method, the metric value is output as an average value for all minibatches. Then even if we have set the frequency to output metrics in job.conf (which means we need the average metric values every several minibatches), the output results are incorrect. Therefore, the previous reset method causes invalid metric value resetting operation.