commit f69811ad202abbd4cb1e18e35b36fd0c74e09d16
author Bobby R. Bruce <bbruce@ucdavis.edu> Thu Feb 18 13:31:22 2021 -0800
committer Bobby R. Bruce <bbruce@ucdavis.edu> Sat Feb 20 06:27:27 2021 +0000
tree 3941494ccccc3531436caff3ade7b3b190e0c454
parent 4dc66b03f1c12e96a7115b1dde3b5ed31433478e

base-stats,python: Update PyBind11 ScalarInfo fields to readonly

This change keeps the ScalarInfo class consistent with the other Info
classes exposed via PyBind11.

Change-Id: I4d420d509e4654de844e75f58aeaaf67109775d3
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/41693
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>

src/python/pybind11/stats.cc

diff --git a/src/python/pybind11/stats.cc b/src/python/pybind11/stats.cc
index e6912e2..1e6773f 100644
--- a/src/python/pybind11/stats.cc
+++ b/src/python/pybind11/stats.cc
@@ -149,9 +149,15 @@
     py::class_<Stats::ScalarInfo, Stats::Info,
                std::unique_ptr<Stats::ScalarInfo, py::nodelete>>(
                    m, "ScalarInfo")
-        .def("value", &Stats::ScalarInfo::value)
-        .def("result", &Stats::ScalarInfo::result)
-        .def("total", &Stats::ScalarInfo::total)
+        .def_property_readonly("value", [](const Stats::ScalarInfo &info) {
+                return info.value();
+            })
+        .def_property_readonly("result", [](const Stats::ScalarInfo &info) {
+                return info.result();
+            })
+        .def_property_readonly("total", [](const Stats::ScalarInfo &info) {
+                return info.total();
+            })
         ;
 
     py::class_<Stats::VectorInfo, Stats::Info,