fix: measured values after the decimal point are truncated #79 #80

hsgwa · 2021-01-20T12:30:59Z

Closes #79
Signed-off-by: hsgwa hasegawa@isp.co.jp

…g#79 Signed-off-by: hsgwa <hasegawa@isp.co.jp>

codecov · 2021-01-20T12:57:00Z

Codecov Report

Merging #80 (c15f041) into master (21d6664) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   24.75%   24.75%           
=======================================
  Files          27       27           
  Lines         820      820           
  Branches      185      185           
=======================================
  Hits          203      203           
  Misses        454      454           
  Partials      163      163

Flag	Coverage Δ
unittests	`24.75% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...opic_statistics_collector/received_message_age.hpp
...rc/libstatistics_collector/collector/collector.cpp
...c_statistics_collector/received_message_period.hpp
...rage_statistics/test_moving_average_statistics.cpp
...ector/moving_average_statistics/moving_average.hpp
...tatistics_collector/topic_statistics_collector.hpp
...istics_collector/test/collector/test_collector.cpp
...statistics_collector/test_received_message_age.cpp
...tistics_collector/test_received_message_period.cpp
...tics_collector/moving_average_statistics/types.cpp
... and 20 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 21d6664...c15f041. Read the comment docs.

emersonknapp

This LGTM - thanks for the patch! The linters are failing for an unrelated reason - fixing it in #81 and then we can update this PR, run CI and go ahead