Skip to content

Commit 6664b2c

Browse files
committed
PR feedback
1 parent 0eeda87 commit 6664b2c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

site/src/comparison.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -211,12 +211,12 @@ impl Metric {
211211
}
212212
}
213213

214-
/// Determines the magntiude of a relative change in percentage terms
214+
/// Determines the magnitude of a percent relative change for a given metric.
215215
///
216216
/// Takes into account how noisy the stat is. For example, instruction
217217
/// count which is normally not very noisy has smaller thresholds than
218218
/// max-rss which can be noisy.
219-
fn absolute_magnitude(&self, change: f64) -> Magnitude {
219+
fn relative_change_magnitude(&self, change: f64) -> Magnitude {
220220
let noise_factor = if self.is_typically_noisy() { 2.0 } else { 1.0 };
221221
let change = change / noise_factor;
222222
if change < 0.2 {
@@ -1110,7 +1110,7 @@ impl TestResultComparison {
11101110
} else {
11111111
Magnitude::VeryLarge
11121112
};
1113-
let absolute_magnitude = self.stat.absolute_magnitude(change * 100.0);
1113+
let absolute_magnitude = self.stat.relative_change_magnitude(change * 100.0);
11141114
fn as_u8(m: Magnitude) -> u8 {
11151115
match m {
11161116
Magnitude::VerySmall => 1,

0 commit comments

Comments
 (0)