addressing comments

aptos-labs · Jan 10, 2025 · 27784fc · 27784fc
1 parent 249af52
commit 27784fc
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 17 deletions.
diff --git a/aptos-move/e2e-benchmark/data/calibration_values.tsv b/aptos-move/e2e-benchmark/data/calibration_values.tsv
@@ -29,5 +29,7 @@ VectorTrimAppend { vec_len: 3000, element_len: 1, index: 100, repeats: 1000 }	11
 VectorTrimAppend { vec_len: 3000, element_len: 1, index: 2990, repeats: 1000 }	119	0.947	1.093	18431.4
 VectorRemoveInsert { vec_len: 3000, element_len: 1, index: 100, repeats: 1000 }	119	0.943	1.107	29034.8
 VectorRemoveInsert { vec_len: 3000, element_len: 1, index: 2998, repeats: 1000 }	119	0.954	1.149	20047.3
+VectorRangeMove { vec_len: 3000, element_len: 1, index: 1000, move_len: 500, repeats: 1000 }	6	0.925	1.001	65311.0
 VectorTrimAppend { vec_len: 100, element_len: 100, index: 0, repeats: 0 }	119	0.909	1.201	293.5
 VectorTrimAppend { vec_len: 100, element_len: 100, index: 10, repeats: 1000 }	119	0.951	1.143	12571.2
+VectorRangeMove { vec_len: 100, element_len: 100, index: 50, move_len: 10, repeats: 1000 }	6	0.925	1.001	7098.0
diff --git a/aptos-move/e2e-benchmark/src/main.rs b/aptos-move/e2e-benchmark/src/main.rs
@@ -206,13 +206,13 @@ fn main() {
             index: 2998,
             repeats: 1000,
         },
-        // EntryPoints::VectorRangeMove {
-        //     vec_len: 3000,
-        //     element_len: 1,
-        //     index: 1000,
-        //     move_len: 500,
-        //     repeats: 1000,
-        // },
+        EntryPoints::VectorRangeMove {
+            vec_len: 3000,
+            element_len: 1,
+            index: 1000,
+            move_len: 500,
+            repeats: 1000,
+        },
         // vectors with large elements
         EntryPoints::VectorTrimAppend {
             // baseline, only vector creation
@@ -227,13 +227,13 @@ fn main() {
             index: 10,
             repeats: 1000,
         },
-        // EntryPoints::VectorRangeMove {
-        //     vec_len: 100,
-        //     element_len: 100,
-        //     index: 50,
-        //     move_len: 10,
-        //     repeats: 1000,
-        // },
+        EntryPoints::VectorRangeMove {
+            vec_len: 100,
+            element_len: 100,
+            index: 50,
+            move_len: 10,
+            repeats: 1000,
+        },
     ];
 
     let mut failures = Vec::new();

diff --git a/crates/transaction-generator-lib/src/publishing/prebuild_packages.rs b/crates/transaction-generator-lib/src/publishing/prebuild_packages.rs
@@ -50,9 +50,9 @@ pub fn create_prebuilt_packages_rs_file(
     writeln!(
         string_buffer,
         "
+use aptos_transaction_generator_lib::entry_point_trait::PreBuiltPackages;carg
 use once_cell::sync::Lazy;
-use std::collections::HashMap;
-use aptos_transaction_generator_lib::entry_point_trait::PreBuiltPackages;",
+use std::collections::HashMap;",
     )
     .expect("Use directive failed");
     writeln!(string_buffer).expect("Empty line failed");

diff --git a/crates/transaction-workloads-lib/src/raw_module_data.rs b/crates/transaction-workloads-lib/src/raw_module_data.rs
@@ -12,9 +12,9 @@
 // This crate should also provide a Rust file that allows proper manipulation of each
 // module defined below.
 
+use aptos_transaction_generator_lib::entry_point_trait::PreBuiltPackages;
 use once_cell::sync::Lazy;
 use std::collections::HashMap;
-use aptos_transaction_generator_lib::entry_point_trait::PreBuiltPackages;
 
 #[rustfmt::skip]
 pub static PACKAGE_SIMPLE_METADATA: Lazy<Vec<u8>> = Lazy::new(|| {