commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t.@apache.org
Subject svn commit: r1311321 - in /commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics: ElitisticListPopulation.java ListPopulation.java
Date Mon, 09 Apr 2012 16:55:03 GMT
Author: tn
Date: Mon Apr  9 16:55:03 2012
New Revision: 1311321

URL: http://svn.apache.org/viewvc?rev=1311321&view=rev
Log:
[MATH-775] remove getChromosomeList method and change internal list to protected.

Modified:
    commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ElitisticListPopulation.java
    commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ListPopulation.java

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ElitisticListPopulation.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ElitisticListPopulation.java?rev=1311321&r1=1311320&r2=1311321&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ElitisticListPopulation.java
(original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ElitisticListPopulation.java
Mon Apr  9 16:55:03 2012
@@ -76,13 +76,13 @@ public class ElitisticListPopulation ext
         ElitisticListPopulation nextGeneration =
                 new ElitisticListPopulation(getPopulationLimit(), getElitismRate());
 
-        final List<Chromosome> oldChromosomes = getChromosomeList();
-        Collections.sort(oldChromosomes);
+        // sort the chromosomes inplace
+        Collections.sort(chromosomes);
 
         // index of the last "not good enough" chromosome
-        int boundIndex = (int) FastMath.ceil((1.0 - getElitismRate()) * oldChromosomes.size());
-        for (int i = boundIndex; i < oldChromosomes.size(); i++) {
-            nextGeneration.addChromosome(oldChromosomes.get(i));
+        int boundIndex = (int) FastMath.ceil((1.0 - getElitismRate()) * chromosomes.size());
+        for (int i = boundIndex; i < chromosomes.size(); i++) {
+            nextGeneration.addChromosome(chromosomes.get(i));
         }
         return nextGeneration;
     }

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ListPopulation.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ListPopulation.java?rev=1311321&r1=1311320&r2=1311321&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ListPopulation.java
(original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math3/genetics/ListPopulation.java
Mon Apr  9 16:55:03 2012
@@ -37,7 +37,7 @@ import org.apache.commons.math3.exceptio
 public abstract class ListPopulation implements Population {
 
     /** List of chromosomes */
-    private List<Chromosome> chromosomes;
+    protected List<Chromosome> chromosomes;
 
     /** maximal size of the population */
     private int populationLimit;
@@ -121,14 +121,6 @@ public abstract class ListPopulation imp
     }
 
     /**
-     * Access the list of chromosomes.
-     * @return the list of chromosomes
-     */
-    protected List<Chromosome> getChromosomeList() {
-        return chromosomes;
-    }
-
-    /**
      * Add the given chromosome to the population.
      * @param chromosome the chromosome to add.
      * @throws NumberIsTooLargeException if the population would exceed the {@code populationLimit}
after



Mime
View raw message