From 651544448d2132d2ed3465c72e9a6dce904d6d3f Mon Sep 17 00:00:00 2001 From: Matthew Treinish Date: Thu, 23 Aug 2018 16:48:12 -0400 Subject: [PATCH] Clean up run_time graph style This commit cleans up the style used for the run_time graph. It switches the individual run_time graph (not the avg or std dev) from a line graph to scatter points. Since there are potentially a lot of data points this makes things cleaner and easier to see. Then the size of those points and the line width are shrunk. When there are a lot of data points things overlap, get cluttered, and hard to see. By shrinking the lines and scatter points it alleviates this to a certain degree. Change-Id: I1271acf10a12df8e36a85e4deacbfc77f7f05bc0 --- subunit2sql/analysis/run_time.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/subunit2sql/analysis/run_time.py b/subunit2sql/analysis/run_time.py index 3fadc9f..2274fea 100644 --- a/subunit2sql/analysis/run_time.py +++ b/subunit2sql/analysis/run_time.py @@ -70,8 +70,8 @@ def generate_series(): ax.xaxis_date() ax.xaxis.set_major_formatter(xfmt) - plt.plot(ts.index, ts, 'k', label='Run Time') - plt.plot(mean.index, mean, 'b', label='Avg. Run Time') + plt.plot(ts.index, ts, 'ko', label='Run Time', markersize=0.45) + plt.plot(mean.index, mean, 'b', label='Avg. Run Time', linewidth=0.45) upper_std_dev = mean + 2 * rolling_std lower_std_dev = mean - 2 * rolling_std # Set negative numbers to 0