-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
regex symbols in get_stats script output #111
Comments
The get_stats.py script generates output similar to the snippet below :
----- snippet ends here -------- In case of long stat names, it is difficult to read the names especially if the user is new to the environment and the stats themselves. |
Hi, Could you please show an example of what it looks like after your fix? Thanks |
Hi, I have modified the update. Here's the complete run output. before the fix:
after the fix:The get_stats.py corresponds to the file with the fix. The original file is named as get_stats.py.orig. At this stage, I replaced the original example_stats.yml with the fixed example_stats.yml (which is the file example_stats.yml.mod here).
Please let me know if you need any additional information. Thanks. |
Hi Since the output of get_stats.py does not contain regex, I added changes to correl_mappings.py so that plot-correlation.py can use the modified output from the fixed get_stats.py/example_stats.yml in order to generate correlation data as before. In order to test the changes to correl_mappings.py, I added a configuration entry corresponding to my GPU device and also a separate stanza for my GPU device to the committed version of correl_mappings.py. The .PR112 file is the one I committed and the .PR112-and-P2200 is the file where I added an entry for my GPU device so that I could verify if the modified correl_mappings.py file allows the correlator script to work fine.
I then copied the correl_mappings.py.PR112-and-P2200 file as correl_mappings.py. The .csv file mentioned in the command line invocation below is the one generated by get_stats.py.
Subsequently, there are files generated in the correl-html/ folder as below:
The above test shows that the modified correl-mappings.py allows the plot-correlation.py script to work fine. Please let me know if you need any additional inputs on the test results or any other clarifications. Thanks |
@JRPan Thanks |
@JRPan Used get_stats.py (the committed version) to generate a csv for the purpose of correlation:
This get_stats.py also uses the modified example_stats.yml. The example_stats.yml used for the test:
The output of the generated csv file is:
The hw run data was generated using the following script: The hw run data is here:
The 22.04.13*csv file does contain data for several stats. An output from the csv.cycle.0 file:
First I modify the correl_mappings.py to exactly match the committed version:
But the output folder is not there. This is expected since there is no matching definition in the correl-mappings.py config_maps.
Now I replace the correl_mappings.py with the one that is the committed version + it includes a mapping for my GPU device, i.e. P2200 Quadro.
In other words, I am able to see that the plot-correlator.py is generating plots when I use the committed correl_mappings.py (+ changes for my GPU device in the correl_mappings.py). Thanks. |
$ python get_stats.py -N AccelWattch-PTX-c139-m5100-100c
The text was updated successfully, but these errors were encountered: