wip

Author: wanguardd

module/move/benchkit/src/analysis.rs

@@ -10,7 +10,15 @@ use std::collections::HashMap;
 pub struct ComparativeAnalysis {
   name: String,
   variants: HashMap<String, Box<dyn FnMut() + Send>>,
-  results: HashMap<String, BenchmarkResult>,
+}
+
+impl std::fmt::Debug for ComparativeAnalysis {
+  fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+    f.debug_struct("ComparativeAnalysis")
+      .field("name", &self.name)
+      .field("variants", &format!("{} variants", self.variants.len()))
+      .finish()
+  }
 }
 
 impl ComparativeAnalysis {
@@ -19,7 +27,6 @@ impl ComparativeAnalysis {
     Self {
       name: name.into(),
       variants: HashMap::new(), 
-      results: HashMap::new(),
     }
   }
 
@@ -59,7 +66,9 @@ impl ComparativeAnalysis {
 /// Report containing results of comparative analysis
 #[derive(Debug)]
 pub struct ComparisonReport {
+  /// Name of the comparison analysis
   pub name: String,
+  /// Results of each algorithm variant tested
   pub results: HashMap<String, BenchmarkResult>,
 }
 
@@ -166,7 +175,9 @@ impl ComparisonReport {
 /// Performance regression analysis
 #[derive(Debug, Clone)]
 pub struct RegressionAnalysis {
+  /// Baseline benchmark results to compare against
   pub baseline_results: HashMap<String, BenchmarkResult>,
+  /// Current benchmark results being analyzed
   pub current_results: HashMap<String, BenchmarkResult>,
 }
 

module/move/benchkit/src/generators.rs

@@ -133,6 +133,7 @@ pub fn generate_urls(size: DataSize) -> Vec<String> {
 }
 
 /// Seeded random data generator using simple LCG
+#[derive(Debug)]
 pub struct SeededGenerator {
   seed: u64,
 }
@@ -183,6 +184,7 @@ pub fn generate_random_vec(size: usize) -> Vec<i32> {
 }
 
 /// Generate test data for common parsing scenarios (based on unilang experience)
+#[derive(Debug)]
 pub struct ParsingTestData;
 
 impl ParsingTestData {

module/move/benchkit/src/measurement.rs

@@ -122,8 +122,11 @@ impl fmt::Display for BenchmarkResult {
 /// Comparison between two benchmark results
 #[derive(Debug, Clone)]
 pub struct Comparison {
+  /// The baseline benchmark result to compare against
   pub baseline: BenchmarkResult,
+  /// The current benchmark result being compared
   pub current: BenchmarkResult,
+  /// Improvement percentage (positive means current is faster than baseline)
   pub improvement_percentage: f64,
 }
 

module/move/benchkit/src/reporting.rs

@@ -190,7 +190,7 @@ impl ReportGenerator {
 
     // Performance tiers
     let fastest = sorted_results.first().unwrap().1;
-    let slowest = sorted_results.last().unwrap().1;
+    let _slowest = sorted_results.last().unwrap().1;
     let median_idx = sorted_results.len() / 2;
     let median = sorted_results[median_idx].1;
 

module/move/benchkit/src/suite.rs

@@ -9,12 +9,24 @@ use std::collections::HashMap;
 
 /// A collection of benchmarks that can be run together
 pub struct BenchmarkSuite {
+  /// Name of the benchmark suite
   pub name: String,
   benchmarks: HashMap<String, Box<dyn FnMut() + Send>>,
   config: MeasurementConfig,
   results: HashMap<String, BenchmarkResult>,
 }
 
+impl std::fmt::Debug for BenchmarkSuite {
+  fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+    f.debug_struct("BenchmarkSuite")
+      .field("name", &self.name)
+      .field("benchmarks", &format!("{} benchmarks", self.benchmarks.len()))
+      .field("config", &self.config)
+      .field("results", &format!("{} results", self.results.len()))
+      .finish()
+  }
+}
+
 impl BenchmarkSuite {
   /// Create a new benchmark suite
   pub fn new(name: impl Into<String>) -> Self {
@@ -103,7 +115,9 @@ impl BenchmarkSuite {
 /// Results from running a benchmark suite
 #[derive(Debug)]
 pub struct SuiteResults {
+  /// Name of the benchmark suite that was run
   pub suite_name: String,
+  /// Individual benchmark results from the suite
   pub results: HashMap<String, BenchmarkResult>,
 }