stats: Add the geometric mean to the histogram

Information
Submitter:	Andreas Hansson
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 9518:aa3cff339427
---------------------------
stats: Add the geometric mean to the histogram

This patch adds the geometric mean to the histogram stat. This requires the
sum of logs of individual samples to be used. This is already calculated in
C++ and therefore simply needs to be passed into the python.

The geometric mean is written to the text based output, whereas only the sum of
logs of individual samples is written to the SQL database. This is done as the
geometric mean can be calculated offline.

Testing Done

Issue Summary

Description	From	Last Updated	Status
salf?	Nilay Vaish	Jan. 28, 2013, 6:41 p.m.	Resolved

Looks good.  Make me think that we should have a harmonic mean for completeness.  Seems like it would be relatively easy to add this to the distribution stat as well.

Sascha Bischoff Jan. 28, 2013, 7:10 p.m. (Jan. 28, 2013, 7:10 p.m.)

Sounds like a good idea. Are you OK with this being added at a later stage in another patch? There are more than enough patches here already.

Nathan Binkert Jan. 29, 2013, 2:44 a.m. (Jan. 29, 2013, 2:44 a.m.)
```
Absolutely.
```

Ship It!

src/python/m5/stats/sql.py (Diff revision 1)

salf?

Show all issues

Sascha Bischoff Jan. 28, 2013, 7:10 p.m. (Jan. 28, 2013, 7:10 p.m.)
```
Fixed. Thanks.
```

Description:

~		Changeset 9494:5744a86325ec
	~	Changeset 9518:aa3cff339427

		stats: Add the geometric mean to the histogram

		This patch adds the geometric mean to the histogram stat. This requires the
		sum of logs of individual samples to be used. This is already calculated in
		C++ and therefore simply needs to be passed into the python.

		The geometric mean is written to the text based output, whereas only the sum of
		logs of individual samples is written to the SQL database. This is done as the
		geometric mean can be calculated offline.

Diff:

Revision 2 (+29 -4)

Show changes

	src/python/m5/stats/__init__.py
	src/python/m5/stats/display.py
	src/python/m5/stats/info.py
	src/python/m5/stats/sql.py

You have a pending review.

Review Board 2.0.15

Screenshots

Files

Issue Summary

Description:

Changeset 9494:5744a86325ec

Changeset 9518:aa3cff339427

Diff: