Skip to content

Commit 93b75eb

Browse files
committed
Fix: out-of-date PMTContinuousObserver readings
Instead of using the power measurement stored in self.results, we should use those in self.parent.results. This will ensure we have the latest readings after continuous benchmarking (benchmark_continuous), instead of the figures measured during the benchmark_default run.
1 parent 4c0a877 commit 93b75eb

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

kernel_tuner/observers/pmt.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,9 +125,7 @@ def after_finish(self):
125125

126126
def get_results(self):
127127
average_kernel_execution_time_ms = self.results["time"]
128-
129-
averages = {key: np.average(values) for key, values in self.results.items()}
130-
self.parent.initialize_results(self.parent.pm_names)
128+
averages = self.parent.get_results()
131129

132130
# correct energy measurement, because current _energy number is collected over the entire duration
133131
# we estimate energy as the average power over the continuous duration times the kernel execution time

0 commit comments

Comments
 (0)