Skip to content

GitLab

Unverified Commit 3d8721a2 authored by Yuting Jiang's avatar Yuting Jiang Committed by GitHub
Browse files

Monitor: Revision - Add 'monitor/' prefix to monitor metrics in result summary (#274) 

**Description**
Add 'monitor/' prefix to monitor metrics in result summary.
parent 6ecded20
Hide whitespace changes
Inline Side-by-side
Showing with 34 additions and 33 deletions
superbench/runner/runner.py
View file @ 3d8721a2
...@@ -336,7 +336,8 @@ def __merge_monitor_metrics(self, node_path): ...@@ -336,7 +336,8 @@ def __merge_monitor_metrics(self, node_path):
for pattern, reduce_type in MonitorRecord.reduce_ops.items(): for pattern, reduce_type in MonitorRecord.reduce_ops.items():
if pattern in metric: if pattern in metric:
reduce_func = Reducer.get_reduce_func(reduce_type) reduce_func = Reducer.get_reduce_func(reduce_type)
metrics_summary[metric] = reduce_func(values) metric_name = 'monitor/{}'.format(metric)
metrics_summary[metric_name] = reduce_func(values)
continue continue
return metrics_summary return metrics_summary
......
tests/runner/test_runner.py
View file @ 3d8721a2
...@@ -244,37 +244,37 @@ def test_merge_monitor_metrics(self): ...@@ -244,37 +244,37 @@ def test_merge_monitor_metrics(self):
"""Test __merge_monitor_metrics.""" """Test __merge_monitor_metrics."""
path = Path('tests/data/monitor/') path = Path('tests/data/monitor/')
expected = { expected = {
'gpu_temperature:0': 50, 'monitor/gpu_temperature:0': 50,
'gpu_temperature:1': 27, 'monitor/gpu_temperature:1': 27,
'gpu_temperature:2': 24, 'monitor/gpu_temperature:2': 24,
'gpu_temperature:3': 26, 'monitor/gpu_temperature:3': 26,
'gpu_temperature:4': 25, 'monitor/gpu_temperature:4': 25,
'gpu_temperature:5': 25, 'monitor/gpu_temperature:5': 25,
'gpu_temperature:6': 23, 'monitor/gpu_temperature:6': 23,
'gpu_temperature:7': 26, 'monitor/gpu_temperature:7': 26,
'gpu_power_limit:0': 250, 'monitor/gpu_power_limit:0': 250,
'gpu_power_limit:1': 200, 'monitor/gpu_power_limit:1': 200,
'gpu_power_limit:2': 250, 'monitor/gpu_power_limit:2': 250,
'gpu_power_limit:3': 250, 'monitor/gpu_power_limit:3': 250,
'gpu_power_limit:4': 250, 'monitor/gpu_power_limit:4': 250,
'gpu_power_limit:5': 250, 'monitor/gpu_power_limit:5': 250,
'gpu_power_limit:6': 250, 'monitor/gpu_power_limit:6': 250,
'gpu_power_limit:7': 250, 'monitor/gpu_power_limit:7': 250,
'gpu_corrected_ecc:0': 12, 'monitor/gpu_corrected_ecc:0': 12,
'gpu_corrected_ecc:1': 0, 'monitor/gpu_corrected_ecc:1': 0,
'gpu_corrected_ecc:2': 0, 'monitor/gpu_corrected_ecc:2': 0,
'gpu_corrected_ecc:3': 0, 'monitor/gpu_corrected_ecc:3': 0,
'gpu_corrected_ecc:4': 0, 'monitor/gpu_corrected_ecc:4': 0,
'gpu_corrected_ecc:5': 0, 'monitor/gpu_corrected_ecc:5': 0,
'gpu_corrected_ecc:6': 0, 'monitor/gpu_corrected_ecc:6': 0,
'gpu_corrected_ecc:7': 0, 'monitor/gpu_corrected_ecc:7': 0,
'gpu_uncorrected_ecc:0': 0, 'monitor/gpu_uncorrected_ecc:0': 0,
'gpu_uncorrected_ecc:1': 0, 'monitor/gpu_uncorrected_ecc:1': 0,
'gpu_uncorrected_ecc:2': 0, 'monitor/gpu_uncorrected_ecc:2': 0,
'gpu_uncorrected_ecc:3': 0, 'monitor/gpu_uncorrected_ecc:3': 0,
'gpu_uncorrected_ecc:4': 0, 'monitor/gpu_uncorrected_ecc:4': 0,
'gpu_uncorrected_ecc:5': 0, 'monitor/gpu_uncorrected_ecc:5': 0,
'gpu_uncorrected_ecc:6': 0, 'monitor/gpu_uncorrected_ecc:6': 0,
'gpu_uncorrected_ecc:7': 0 'monitor/gpu_uncorrected_ecc:7': 0
} }
self.assertEqual(self.runner._SuperBenchRunner__merge_monitor_metrics(path), expected) self.assertEqual(self.runner._SuperBenchRunner__merge_monitor_metrics(path), expected)
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment