Closed HeuristicLab-Trac-Bot closed 11 years ago
- Caches shall be directly in the
(Timeframe-)FeatureCorrelationView
- Caches should use
Tuple<T1, T2>
instead of nested dictionariesAbstractFeatureCorrelationView
shall inherit fromStringConvertibleMatrixView
to reduce code duplication- Replace
HeatMap
variable 'currentCorrelation' withdouble[,]
- add a text box to
TimeframeCorrelationView
to input how many time frames shall be calculated (remove combo box which is currently used)
- add an interface for dependency correlation calculators
- removed combo box in
TimeframeCorrelationView
and added a textbox instead- caches are directly in
(Timeframe-)FeatureCorrelationView
- caches use
Tuple<>
instead of nested dictionaries- a control
EnhancedStringConvertibleMatrix
inherits fromStringConvertibleMatrixView
to reduce code duplication- add interface
IDependencyCalculator
to several calculators- fixed bug: a previous started calculation is cancelled, if a new calculation shall be started and the values are already in the cache
- fixed bug: if the content is changed, the calculation is cancelled
HeatMap
is still used for the dependency representation, because a class is needed which implementsIStringConvertibleMatrix
and it has a maximum and minimum value.
r8834: changed
CalculateHoeffdingsDTest
due to a change in the name of a static method
- put
IDependencyCalculator
s in own directory- changed
DoubleRange Interval
todouble Minimum\Maximum
inIDependencyCalculator
AbstractFeatureCorrelationView
now usesDoubleMatrix
instead ofHeatMap
- corrected displaying of the Text of
StringConvertibleMatrixVisibilityDialog
and its subclasses- corrected sorting of
EnhancedStringConvertibleMatrixView
Reviewing comments:
- Unify names of controls that are used in views (e.g.,
CorrelationCalcLabel
,minimumLabel
,HeatMapProgressBar
)- TimeFrameFeatureCorrelationView:
- TimeFrameTextBox should check for valid values in the validating event
- The correlation should be recalculated after the validated event of the TimeFrameTextBox was triggered. Currently it does not react on TAB.
- Move partitions from the
FeatureCorrelationHelper
to theAbstractFeatureCorrelationView
and use indexes in the bwInfo.- Remove *.resx files from repository.
- Add missing license headers (.designer files).
- Show / Hide rows throws an exception during dialog creation for the symbolic classification example.
- The selected variable for the timeframe correlation should be the target variable per default.
r8880: implemented changes suggested by mkommend in comment:49:#1292 [[BR]] r8881: renamed some controls
Also corrected
DeregisterContentEvents
inAbstractFeatureCorrelationView
. AbstractFeatureCorrelationView.resx can't be removed, because it contains an image of the possible colors for the correlation.
Issue migrated from trac ticket # 1292
milestone: HeuristicLab 3.3.8 | component: Problems.DataAnalysis.Views | priority: medium | resolution: done
2010-11-22 15:57:45: @mkommend created the issue