Skip to content
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

Small improvements to python scripts #23

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

rejunity
Copy link

@rejunity rejunity commented May 12, 2017

Changes are aimed to reduce additional steps when benchmarking on a fresh setup and makes it easier to update README.md file.

  1. Automatised cuDDN version conversion from int to a friendly string. Will automatically convert 5105 to '5.1.05'. Implemented as cudnn_name() function instead of cudnn_map dict.
  2. Formatted printed out tables so that they are more readable in text format. That does not affect .md formatting since .md parser strips whitespace.
BEFORE:

|Maxwell Titan X|4.0.07|150.90|268.64|419.54|
|Pascal Titan X|None|238.04|371.40|609.43|
|GTX 1080 Ti|None|225.36|368.42|593.79|
|GTX 1080|None|299.05|461.67|760.72|
|Maxwell Titan X|None|382.39|583.83|966.22|
|CPU: Dual Xeon E5-2630 v3|None|6572.17|10300.61|16872.78|

NOW:

|Maxwell Titan X          |4.0.07 | 150.90| 268.64| 419.54|
|GTX 1080 Ti              |None   | 225.36| 368.42| 593.79|
|Pascal Titan X           |None   | 238.04| 371.40| 609.43|
|GTX 1080                 |None   | 299.05| 461.67| 760.72|
|Maxwell Titan X          |None   | 382.39| 583.83| 966.22|
|CPU: Dual Xeon E5-2630 v3|None   |6572.17|10300.61|16872.78|
  1. Made order of printed out models to match order in README.md. Order is defined by model_names_sorted array in analyze_cnn_benchmark_results.py file
  2. Automatically create output directory, if missing.
  3. Changed DEFAULT_MODELS to match file structure in models.zip

@rejunity rejunity mentioned this pull request Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant