From 0f5cff762fc918fa203c01b900bb2dc3c6e0c4db Mon Sep 17 00:00:00 2001 From: Pascal Seitz Date: Tue, 4 Oct 2022 12:30:19 +0800 Subject: [PATCH] move enumerate and remove computation --- fastfield_codecs/src/linear.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/fastfield_codecs/src/linear.rs b/fastfield_codecs/src/linear.rs index 2926f45fc..6ed2d84b6 100644 --- a/fastfield_codecs/src/linear.rs +++ b/fastfield_codecs/src/linear.rs @@ -131,9 +131,8 @@ impl FastFieldCodec for LinearCodec { let num_samples = 100; let step_size = (limit_num_vals / num_samples).max(1); // 20 samples let mut sample_positions_and_values: Vec<_> = Vec::new(); - for (idx, val) in column.iter().step_by(step_size as usize).enumerate() { - let pos = idx as u64 * step_size; - sample_positions_and_values.push((pos, val)); + for (pos, val) in column.iter().enumerate().step_by(step_size as usize) { + sample_positions_and_values.push((pos as u64, val)); } let line = Line::estimate(&sample_positions_and_values);