Changeset 1247 for cpp/frams/genetics/genooperators.cpp

Timestamp:

05/21/23 22:52:02 (2 years ago)

Author:

Maciej Komosinski

Message:

Cosmetic

File:

• cpp/frams/genetics/genooperators.cpp (modified) (2 diffs)

cpp/frams/genetics/genooperators.cpp ¶

@@ -4 +4 @@
 
 #include <ctype.h>  //isupper()
+#include <algorithm> // std::min, std::max
 #include "genooperators.h"
 #include <common/log.h>
 #include <common/nonstd_math.h>
 #include <frams/util/rndutil.h>
-#include <algorithm> // std::min, std::max
 
 //
@@ -537 +537 @@
 string GenoOperators::simplifiedModifiers(const string & original)
 {
-        const int MAX_NUMBER_SAME_TYPE = 6; // max. number of modifiers of each type (case-insensitive). rR is treated separately in simplification because their influence follows different (i.e., simple additive) logic - so the simplifiedModifiersFixedOrder() logic with cancelling out antagonistic modifiers is appropriate for rR.
+        const int MAX_NUMBER_SAME_TYPE = 5; // max. number of modifiers of each type (case-insensitive). The more characters, the closer we can get to min and max values of a given property at the expense of the length of evolved genotypes. 5 is "close enough", but how close we get to the extreme also depends on the initial value of a given property, which is not always exactly in the middle of min and max. rR is treated separately in simplification because their influence follows different (i.e., simple additive) logic - so the simplifiedModifiersFixedOrder() logic with cancelling out antagonistic modifiers would be appropriate for rR.
         int counter[256] = {}; //initialize with zeros; 256 is unnecessarily too big and redundant, but enables very fast access (indexed directly by the ascii code)
         string simplified = "";