Drop assertion on even population size in GeneticSolver::new

Closes #479
ecrs-org · May 1, 2024 · a2abda5 · a2abda5
1 parent a380320
commit a2abda5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/ga.rs b/src/ga.rs
@@ -242,8 +242,6 @@ where
     pub fn new(
         config: GAConfig<IndividualT, MutOpT, CrossOpT, SelOpT, ReplOpT, PopGenT, FitnessT, ProbeT>,
     ) -> Self {
-        assert_eq!(config.params.population_size % 2, 0); // Required for most of operators right
-                                                          // now
         GeneticSolver {
             config,
             metadata: Metrics::new(None, None, 0),

diff --git a/src/ga/operators/selection.rs b/src/ga/operators/selection.rs
@@ -26,12 +26,12 @@ pub trait SelectionOperator<IndividualT: IndividualTrait> {
     ///
     /// ### Arguments
     ///
-    /// * `metadata` - [crate::ga::GAMetadata] information on current stage of the algorithm (iteration, elapsed time, etc.)
+    /// * `metrics` - [crate::ga::Metrics] information on current stage of the algorithm (iteration, elapsed time, etc.)
     /// * `population` - individuals to choose mating pool from
     /// * `count` - target number of individuals in mating pool
     fn apply<'a>(
         &mut self,
-        metadata: &Metrics,
+        metrics: &Metrics,
         population: &'a [IndividualT],
         count: usize,
     ) -> Vec<&'a IndividualT>;