power_dashboard: Don't remove first data point
We use power_telemetry_utils.start_measurement() now, so
the first data point is now valid.
BUG=b:121157449
TEST=ran power_Idle / check dashboard.
Change-Id: I938e1c517020c405f5e977a6cef440bfa8794b98
Signed-off-by: Puthikorn Voravootivat <puthik@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1775097
Reviewed-by: Mengqi Guo <mqg@chromium.org>
diff --git a/client/cros/power/power_dashboard.py b/client/cros/power/power_dashboard.py
index f8f0a3e..809a63c 100644
--- a/client/cros/power/power_dashboard.py
+++ b/client/cros/power/power_dashboard.py
@@ -414,7 +414,7 @@
return None
power_dict = collections.defaultdict(dict, {
- 'sample_count': len(self._logger.readings) - 1,
+ 'sample_count': len(self._logger.readings),
'sample_duration': 0,
'average': dict(),
'data': dict(),
@@ -422,7 +422,7 @@
if power_dict['sample_count'] > 1:
total_duration = self._logger.times[-1] - self._logger.times[0]
power_dict['sample_duration'] = \
- 1.0 * total_duration / power_dict['sample_count']
+ 1.0 * total_duration / (power_dict['sample_count'] - 1)
self._create_padded_domains()
for i, domain_readings in enumerate(zip(*self._logger.readings)):
@@ -430,8 +430,7 @@
domain = self._padded_domains[i]
else:
domain = self._logger.domains[i]
- # Remove first item because that is the log before the test begin.
- power_dict['data'][domain] = domain_readings[1:]
+ power_dict['data'][domain] = domain_readings
power_dict['average'][domain] = \
numpy.average(power_dict['data'][domain])
if self._unit: