Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/41754 )
Change subject: base-stats,python: Add Units to the Python Stats
......................................................................
base-stats,python: Add Units to the Python Stats
Change-Id: Ic8d3c9a5c2bb7fbe51b8672b74b0e5fb17906a5e
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/41754
Tested-by: kokoro <[email protected]>
Reviewed-by: Bobby R. Bruce <[email protected]>
Maintainer: Bobby R. Bruce <[email protected]>
---
M src/python/m5/stats/gem5stats.py
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/python/m5/stats/gem5stats.py
b/src/python/m5/stats/gem5stats.py
index 3afc776..9446045 100644
--- a/src/python/m5/stats/gem5stats.py
+++ b/src/python/m5/stats/gem5stats.py
@@ -148,7 +148,7 @@
def __get_scaler(statistic: _m5.stats.ScalarInfo) -> Scalar:
value = statistic.value
- unit = None # TODO https://gem5.atlassian.net/browse/GEM5-850.
+ unit = statistic.unit
description = statistic.desc
# ScalarInfo uses the C++ `double`.
datatype = StorageType["f64"]
@@ -161,7 +161,7 @@
)
def __get_distribution(statistic: _m5.stats.DistInfo) -> Distribution:
- unit = None # TODO https://gem5.atlassian.net/browse/GEM5-850.
+ unit = statistic.unit
description = statistic.desc
value = statistic.values
bin_size = statistic.bucket_size
@@ -198,7 +198,7 @@
for index in range(statistic.size):
# All the values in a Vector are Scalar values
value = statistic.value[index]
- unit = None # TODO https://gem5.atlassian.net/browse/GEM5-850.
+ unit = statistic.unit
description = statistic.subdescs[index]
# ScalarInfo uses the C++ `double`.
datatype = StorageType["f64"]
4 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/41754
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ic8d3c9a5c2bb7fbe51b8672b74b0e5fb17906a5e
Gerrit-Change-Number: 41754
Gerrit-PatchSet: 7
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
