Free cookie consent management tool by TermsFeed Policy Generator

Changes between Version 1 and Version 2 of ReviewHeuristicLab3.3.0CodeSelection


Ignore:
Timestamp:
03/19/10 12:58:15 (14 years ago)
Author:
abeham
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • ReviewHeuristicLab3.3.0CodeSelection

    v1 v2  
    22
    33== Reviewer: abeham ==
    4  * `Selector` contains the parameters !NumberOfSelectedSubScopes and !CopySelected, but does not actually use this parameter in any way. `Selector` should not implement `ISelector`, all derived classes that are selectors in the sense of `ISelector` should implement this interface instead. Currently `TabuSelector` and probably future custom selectors appear as selectors in the SGA. It is true that `TabuSelector` is a selector (divides subscopes in remaining and selected), but not an `ISelector`.
     4 * `Selector` should not implement `ISelector`, all derived classes that are selectors in the sense of `ISelector` should implement this interface instead. Currently `TabuSelector` and probably future custom selectors appear as selectors in the SGA. It is true that `TabuSelector` is a selector (divides subscopes in remaining and selected), but not an `ISelector`.
     5  * A related note: Currently `TabuSelector` is the only "Selector" that appears twice in the !SelectorParameter of the SGA.
    56
    67----