stdlib: Improve gem5 PyStats #996
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BobbyRBruce
added
stats
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
4 times, most recently
from
a9665f0
to
d842e35
Compare
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
7 times, most recently
from
fd4a537
to
261c501
Compare
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
4 times, most recently
from
ee2d4f0
to
7fa7584
Compare
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
7 times, most recently
from
e893be9
to
a1c6b37
Compare
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
2 times, most recently
from
7dd21a7
to
515dac4
Compare
Harshil2107
force-pushed
the
vector-fixes-and-improvements
branch
from
515dac4
to
350ef31
Compare
This isn't a true Base class, it's just a Vector. In gem5 all Vectors are Scalar Vectors. This change simplfies the naming. Change-Id: Ib8881d854ab18de6acbf0fb200db2de6a43621a7
The big thing missing from the Vector stats was that each position in the vector could have it's own unique id (a str, float, or int) and each position in the vector can have its own description. Therefore, to add this the Vector is represented as a dictionary mapping the unique ID to a Pystat Scaler (whcih can have it's own unique description. Change-Id: I3a8634f43298f6491300cf5a4f9d25dee8101808
As Distribution inherits from Vector, it should be constructed with a Dictionary of scalars (in our implementation, a dictionary mapping the vector position's unique id for each bin and the value of that bin). Change-Id: Ie603c248e5db4b6dd7f71cc453eebd78793f69a3
This appears to have no equivalent type in the CPP stats and was never utilized in PyStats. Change-Id: Ia9afc83b4159eb1ab2c6f44ec0ad86cd73f2a4f8
Change-Id: Icb2f691abf88ef4bac8d277e421329edb000209b
This was not used and easily confused with the other 'Vector' in PyStats. Change-Id: I9294bb0ae04db0537c87a5f50ce023fc83d587b8
This is inclusive of tests to ensure they have implemented correctly. Change-Id: I5c84d5ffdb7b914936cfd86ca012a7b141eeaf42
Change-Id: Iba0c93ffa5c4b18acf75af82965c63a8881df189
1. Thests here for the Scalar tasks are named appropriately. Not just generic "SimStats tess". 2. We remove 'simstat' terminology. The correct word is "Pystats". Change-Id: Idebc4e750f4be7f140ad6bff9c6772f580a24861
Change-Id: I1d93453072d12aa2dd40066f364723de1225b4e0
Change-Id: Iae12a0ac88f9646acb00e73d70f83b1e2ff94ac9
BobbyRBruce
force-pushed
the
vector-fixes-and-improvements
branch
from
350ef31
to
0f6bd24
Compare
|
I believe I've fixed the failing tests. This should be ready for review. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR incorporates numerous improvements and fixes to the gem5 PyStats. This includes:
simobjectvec[0]. (This replaces theVectorgroup PyStat)SparseHistPyStats.Vector2dto PyStats.DistributionPyStats is fixed to be a vector of Scalars.