Satisfy clippy in gas-price-analysis

crates/fuel-gas-price-algorithm/gas-price-analysis/src/charts.rs

@@ -80,11 +80,10 @@ pub fn draw_gas_prices(
     da_gas_prices: &[u64],
     title: &str,
 ) -> anyhow::Result<()> {
-    let gas_prices_gwei: Vec<_> = gas_prices.into_iter().map(|x| x / ONE_GWEI).collect();
+    let gas_prices_gwei: Vec<_> = gas_prices.iter().map(|x| x / ONE_GWEI).collect();
     let _exec_gas_prices_gwei: Vec<_> =
-        _exec_gas_prices.into_iter().map(|x| x / ONE_GWEI).collect();
-    let da_gas_prices_gwei: Vec<_> =
-        da_gas_prices.into_iter().map(|x| x / ONE_GWEI).collect();
+        _exec_gas_prices.iter().map(|x| x / ONE_GWEI).collect();
+    let da_gas_prices_gwei: Vec<_> = da_gas_prices.iter().map(|x| x / ONE_GWEI).collect();
     // const GAS_PRICE_COLOR: RGBColor = BLACK;
     // const EXEC_GAS_PRICE_COLOR: RGBColor = RED;
     const DA_GAS_PRICE_COLOR: RGBColor = BLUE;
@@ -146,7 +145,7 @@ pub fn draw_gas_prices(
 
 pub fn draw_fullness(
     drawing_area: &DrawingArea<BitMapBackend, Shift>,
-    fullness: &Vec<(u64, u64)>,
+    fullness: &[(u64, u64)],
     title: &str,
 ) -> anyhow::Result<()> {
     const FULLNESS_COLOR: RGBColor = BLACK;
@@ -267,16 +266,14 @@ pub fn draw_profit(
     const ACTUAL_PROFIT_COLOR: RGBColor = BLACK;
     const PROJECTED_PROFIT_COLOR: RGBColor = RED;
     const PESSIMISTIC_BLOCK_COST_COLOR: RGBColor = BLUE;
-    let actual_profit_gwei: Vec<_> = actual_profit
-        .into_iter()
-        .map(|x| x / ONE_GWEI as i128)
-        .collect();
+    let actual_profit_gwei: Vec<_> =
+        actual_profit.iter().map(|x| x / ONE_GWEI as i128).collect();
     let projected_profit_gwei: Vec<_> = projected_profit
-        .into_iter()
+        .iter()
         .map(|x| x / ONE_GWEI as i128)
         .collect();
     let pessimistic_block_costs_gwei: Vec<_> = pessimistic_block_costs
-        .into_iter()
+        .iter()
         .map(|x| x / ONE_GWEI as u128)
         .collect();
     let min = *std::cmp::min(

crates/fuel-gas-price-algorithm/gas-price-analysis/src/simulation.rs

@@ -87,7 +87,7 @@ impl Simulator {
         // Scales the gas price internally, value is arbitrary
         let gas_price_factor = 100;
         let always_normal_activity = L2ActivityTracker::new_always_normal();
-        let updater = AlgorithmUpdaterV1 {
+        AlgorithmUpdaterV1 {
             min_exec_gas_price: 10,
             min_da_gas_price: 10,
             // Change to adjust where the exec gas price starts on block 0
@@ -114,11 +114,10 @@ impl Simulator {
             last_profit: 0,
             second_to_last_profit: 0,
             l2_activity: always_normal_activity,
-        };
-        updater
+        }
     }
 
-    fn execute_simulation<'a>(
+    fn execute_simulation(
         &self,
         capacity: u64,
         max_block_bytes: u64,
@@ -161,7 +160,7 @@ impl Simulator {
 
             // Update DA blocks on the occasion there is one
             if let Some((range, cost)) = da_block {
-                for height in range.to_owned() {
+                for height in range {
                     updater
                         .update_da_record_data(height..(height + 1), cost)
                         .unwrap();
@@ -178,7 +177,7 @@ impl Simulator {
         let bytes_and_costs: Vec<_> = bytes
             .iter()
             .zip(self.da_cost_per_byte.iter())
-            .map(|(bytes, da_cost_per_byte)| (*bytes, (*bytes * da_cost_per_byte) as u64))
+            .map(|(bytes, da_cost_per_byte)| (*bytes, *bytes * da_cost_per_byte))
             .collect();
 
         let actual_profit: Vec<i128> = actual_costs
@@ -209,7 +208,7 @@ impl Simulator {
         &self,
         da_recording_rate: usize,
         da_finalization_rate: usize,
-        fullness_and_bytes: &Vec<(u64, u64)>,
+        fullness_and_bytes: &[(u64, u64)],
     ) -> Vec<Option<(Range<u32>, u128)>> {
         let l2_blocks_with_no_da_blocks =
             std::iter::repeat(None).take(da_finalization_rate);

crates/fuel-gas-price-algorithm/gas-price-analysis/src/simulation/da_cost_per_byte.rs

@@ -18,8 +18,7 @@ pub fn get_da_cost_per_byte_from_source(
             let original = get_costs_from_csv_file(&file_path, sample_size);
             original
                 .into_iter()
-                .map(|x| std::iter::repeat(x).take(update_period))
-                .flatten()
+                .flat_map(|x| iter::repeat(x).take(update_period))
                 .collect()
         }
     }
@@ -89,8 +88,7 @@ fn arbitrary_cost_per_byte(size: usize, update_period: usize) -> Vec<u64> {
         .map(|x| x * ROUGH_COST_AVG + ROUGH_COST_AVG) // Sine wave is between -1 and 1, scale and shift
         .map(|x| x as u64)
         .map(|x| std::cmp::max(x, 1))
-        .map(|x| iter::repeat(x).take(update_period as usize))
-        .flatten()
-        .take(size as usize)
+        .flat_map(|x| iter::repeat(x).take(update_period))
+        .take(size)
         .collect()
 }