Free cookie consent management tool by TermsFeed Policy Generator

Changes between Version 2 and Version 3 of Ticket #2175, comment 26


Ignore:
Timestamp:
11/19/15 17:31:57 (9 years ago)
Author:
gkronber
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2175, comment 26

    v2 v3  
    11Review comments:
    2  - Why is the tree simplified twice in the SymbolicDataAnalysisExpressionTreeSimplificationOperator?
     2 - Why is the tree simplified twice in the SymbolicDataAnalysisExpressionTreeSimplificationOperator? 
    33 - I think it is more readable if cases that are treated equally in the ComplexityCalculater have 'fall-through' cases in the switch statement.
    44 - The calculation of the complexity for the power and root symbol should be explained (in a comment?)
     5 - Why have properties TrainingBestSolutions and TrainingBestsolutionQualities in ...MultiObjectiveTrainingBestSolutionAnalyzer been changed from public to private? The values are accessible anyway through the parameter properties.
    56
    67r13300: made some changes while reviewing the code
     8r13307: fixed a typo while reviewing