From 5ee2c2aff28b134cd1feadec1bcb5d61d29f00f2 Mon Sep 17 00:00:00 2001 From: Nelson Chen Date: Sun, 15 Nov 2015 14:56:37 -0800 Subject: [PATCH] Latency test is not ping time for the purposes of display Reference: https://github.com/sivel/speedtest-cli/pull/199 --- src/speedtest.rs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/speedtest.rs b/src/speedtest.rs index 061fd45..b4f7c26 100644 --- a/src/speedtest.rs +++ b/src/speedtest.rs @@ -308,9 +308,12 @@ pub fn get_best_server_based_on_latency(servers: &[SpeedTestServer]) info!("Sampled {} ms", latency_measurement.num_milliseconds()); latency_measurements.push(latency_measurement); } + // Divide by the double to get the non-RTT time but the trip time. + // NOT PING or RTT + // https://github.com/sivel/speedtest-cli/pull/199 let latency = latency_measurements.iter().fold(Duration::zero(), |a, &i| a + i) / - latency_measurements.iter().count() as i32; - info!("Averaged to {} ms", latency.num_milliseconds()); + (latency_measurements.iter().count() as i32) * 2; + info!("Trip calculated to {} ms", latency.num_milliseconds()); if latency < fastest_latency { fastest_server = Some(server);