Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

Commit 1f21118

Browse files
committed
Fix the number of percentile dots
5 % and 95 % got to both a dot and a whisker
1 parent b7ea60c commit 1f21118

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

metrics/report/report_dockerfile/tidy_scaling_nc.R

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,13 +95,13 @@ if (length(latencydata[[1]]) <= 5 || length(unique(latencydata$testname)) > 1) {
9595
}
9696
}
9797
# Boxplot (above) covers at most 5 percentiles around the center (median).
98-
# Visualize the rest by points
98+
# Visualize the rest using a point for each percentile.
9999
if (perc_count > 5) {
100-
for (n in seq(1, perc_count-6)) {
100+
for (n in seq(1, (perc_count-5)/2)) {
101101
lower_name=names(ltpt)[n+1]
102102
upper_name=names(ltpt)[perc_count-n+2]
103-
latp = latp + geom_point(aes_string(y=lower_name, color="testname"))
104-
latp = latp + geom_point(aes_string(y=upper_name, color="testname"))
103+
latp = latp + geom_point(aes_string(group="interaction(testname,n_pods)",y=lower_name, color="testname"))
104+
latp = latp + geom_point(aes_string(group="interaction(testname,n_pods)",y=upper_name, color="testname"))
105105
}
106106
}
107107
} else {

0 commit comments

Comments
 (0)