Skip to content

Commit 9314454

Browse files
Merge pull request #1071 from Mark-Simulacrum/reduce-log
Reduce log noise from comparison logic
2 parents e92928f + 65bb561 commit 9314454

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

site/src/comparison.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ use crate::load::SiteCtxt;
99
use crate::selector::{self, Tag};
1010

1111
use collector::Bound;
12-
use log::debug;
1312
use serde::Serialize;
1413

1514
use std::collections::{HashMap, HashSet};
@@ -627,7 +626,7 @@ impl BenchmarkVariances {
627626
variance_data.retain(|_, v| v.data.len() >= Self::MIN_PREVIOUS_COMMITS);
628627

629628
for ((bench, _, _), results) in variance_data.iter_mut() {
630-
debug!("Calculating variance for: {}", bench);
629+
log::trace!("Calculating variance for: {}", bench);
631630
results.calculate_description();
632631
}
633632

@@ -725,7 +724,7 @@ impl BenchmarkVariance {
725724
let percent_significant_changes = ((percent_changes.len() - non_significant.len()) as f64
726725
/ percent_changes.len() as f64)
727726
* 100.0;
728-
debug!(
727+
log::trace!(
729728
"Percent significant changes: {:.1}%",
730729
percent_significant_changes
731730
);
@@ -737,7 +736,7 @@ impl BenchmarkVariance {
737736

738737
let delta_mean =
739738
non_significant.iter().cloned().sum::<f64>() / (non_significant.len() as f64);
740-
debug!("Ratio change: {:.4}", delta_mean);
739+
log::trace!("Ratio change: {:.4}", delta_mean);
741740
if delta_mean > Self::NOISE_THRESHOLD {
742741
self.description = BenchmarkVarianceDescription::Noisy;
743742
}

0 commit comments

Comments
 (0)