> On Jan. 25, 2013, 9:05 a.m., Nathan Binkert wrote: > > src/python/m5/stats/display.py, line 222 > > <http://reviews.gem5.org/r/1636/diff/1/?file=33198#file33198line222> > > > > Why is this in this commit? doesn't seem like a bugfix to me. What > > problem is this fixing?
I have split this into a second patch. It was trying to address issues with formulas not being printed in the same way as in the old C++ stats system. - Sascha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1636/#review3898 ----------------------------------------------------------- On Jan. 29, 2013, 2:13 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1636/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2013, 2:13 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9510:a32b2d4dcb49 > --------------------------- > stats: Add documentation to the python statistics system > > Adds documentation to the python statistics which previously had > almost no documentation. This documentation is in doxygen style and is > therefore included in the standard doxygen documentation for gem5. The > path to the python stats system has been included in the doxygen > search path. > > > Diffs > ----- > > src/Doxyfile 5532a1642108 > src/python/m5/stats/__init__.py 5532a1642108 > src/python/m5/stats/context.py PRE-CREATION > src/python/m5/stats/display.py PRE-CREATION > src/python/m5/stats/info.py PRE-CREATION > > Diff: http://reviews.gem5.org/r/1636/diff/ > > > Testing > ------- > > Ran a number of simulations > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
