Skip to content

Commit

Permalink
Address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
VirajP1002 committed Aug 21, 2024
1 parent 5c4d3c8 commit d0942b7
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 4 deletions.
8 changes: 4 additions & 4 deletions scripts/benchmark_stats.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,11 @@ def __str__(self):
f"Percentile Averages:\n"
f"{formatted_percentiles}\n"
f"---\n"
f"GETs (99th): {formatted_metrics.get('average_get')}\n"
f"POSTs (99th): {formatted_metrics.get('average_post')}\n"
f"GETs (99th): {formatted_metrics['average_get']}\n"
f"POSTs (99th): {formatted_metrics['average_post']}\n"
f"---\n"
f"PDF: {formatted_metrics.get('pdf_percentile')}\n"
f"Session: {formatted_metrics.get('session_percentile')}\n"
f"PDF: {formatted_metrics['pdf_percentile']}\n"
f"Session: {formatted_metrics['session_percentile']}\n"
f"---\n"
f"Total Requests: {self.total_requests:,}\n"
f"Total Failures: {self._total_failures:,}\n"
Expand Down
5 changes: 5 additions & 0 deletions scripts/visualise_results.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ def plot_data(dataframes, number_of_days_to_plot):
fig, axes = plt.subplots(nrows=1, ncols=2, figsize=(15, 6))

for i in range(len(dataframes)):
"""
We use the .subplot() method below to switch the indexes of the plots themselves,
if we do not switch plots or remove this method, the visuals (such as the graph background)
and, most importantly, the axes values will only be applied to the second subplot.
"""
plt.subplot(1, 2, i + 1)
if (
number_of_days_to_plot and number_of_days_to_plot <= 45
Expand Down

0 comments on commit d0942b7

Please sign in to comment.