diff options
| author | Jakub Beránek <berykubik@gmail.com> | 2025-04-07 12:52:09 +0200 |
|---|---|---|
| committer | Jakub Beránek <berykubik@gmail.com> | 2025-04-07 12:52:09 +0200 |
| commit | 1a4863421bcae3ca1d50490256f4cd9236dc49cd (patch) | |
| tree | 9afaed75b65ce1ba990f89e737ac08132ea31f37 | |
| parent | b9856b6e400709392dd14599265b6fd52fc19f3e (diff) | |
| download | rust-1a4863421bcae3ca1d50490256f4cd9236dc49cd.tar.gz rust-1a4863421bcae3ca1d50490256f4cd9236dc49cd.zip | |
Sort job duration changes by absolute duration
It was supposed to be like this from the start, but I forgot to apply the `abs` operation, as I got sidetracked with how to actually compare floats...
| -rw-r--r-- | src/ci/citool/src/analysis.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ci/citool/src/analysis.rs b/src/ci/citool/src/analysis.rs index 7fbfad467c6..6ac485b12ba 100644 --- a/src/ci/citool/src/analysis.rs +++ b/src/ci/citool/src/analysis.rs @@ -225,7 +225,7 @@ pub fn output_largest_duration_changes(job_metrics: &HashMap<JobName, JobMetrics }); } } - changes.sort_by(|e1, e2| e1.change.partial_cmp(&e2.change).unwrap().reverse()); + changes.sort_by(|e1, e2| e1.change.abs().partial_cmp(&e2.change.abs()).unwrap().reverse()); println!("# Job duration changes"); for (index, entry) in changes.into_iter().take(10).enumerate() { |