Skip to content

Commit

Permalink
Merge pull request #283 from wvbbreu/fix-pmt-continuous-observer
Browse files Browse the repository at this point in the history
Fix: out-of-date PMTContinuousObserver readings
  • Loading branch information
benvanwerkhoven authored Dec 5, 2024
2 parents 6ebf773 + 93b75eb commit 083a3ee
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions kernel_tuner/observers/pmt.py
Original file line number Diff line number Diff line change
Expand Up @@ -125,9 +125,7 @@ def after_finish(self):

def get_results(self):
average_kernel_execution_time_ms = self.results["time"]

averages = {key: np.average(values) for key, values in self.results.items()}
self.parent.initialize_results(self.parent.pm_names)
averages = self.parent.get_results()

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

0 comments on commit 083a3ee

Please sign in to comment.