Opened 2 months ago

Last modified 7 weeks ago

#2809 accepted enhancement

Remove "Logic" Classes in DataPreprocessing

Reported by: pfleck Owned by: pfleck
Priority: medium Milestone: HeuristicLab 3.3.16
Component: DataPreprocessing Version: branch
Keywords: Cc:

Description

The "Logics" in the DataPreprocessing encapsulates different aspects of how the DataPreprocessingData can be accessed and manipulated. However, the "Logic"-classes mostly contain some helper methods that are only used within a single corresponding view. The logic classes should be removed and the code moved to the corresponding views.

By removing the "Logics", the initialization of the DataPreprocessingView should be simplified to avoid the highly coupled and entangled object initialization. Generally, the overall design of the data preprocessing should be more like the rest of HL.

Change History (9)

comment:1 Changed 2 months ago by pfleck

  • Status changed from new to accepted

comment:2 Changed 2 months ago by pfleck

r15268 Added branch with the DataPreprocessing plugins.

comment:3 Changed 2 months ago by pfleck

r15269: Removed SearchLogic

comment:4 Changed 2 months ago by pfleck

r15270: Removed TransactionalPreprocessingData and moved relevant code to PreprocessingData.

comment:5 Changed 2 months ago by pfleck

r15274:

  • Removed FilterLogic.
  • Made Contents storable and implemented proper cloning.

comment:6 Changed 2 months ago by pfleck

r15283: Removed StatisticsLogic.

comment:7 Changed 2 months ago by pfleck

r15285 removed ManipulationLogic

  • Moved "ReplaceByAverage/Random/..." to DataGridContent only (functions are redundant in manipulation view)
    • Refactored some code for replacing values with ...
  • Removed Smoothing (not very usefull and changes the data incomprehensibly)
  • Moved Shuffling to DataGridContent
  • Moved "Delete Rows/Columns with insufficient ..." to ManipulationContent
  • Removed Logic Folder
    • Moved Filter folder out

comment:8 Changed 2 months ago by pfleck

r15291: Added (Double/String/DateTime)PreprocessingDataColumn. (experimental state)

comment:9 Changed 7 weeks ago by pfleck

r15309 Worked on type-save PreprocessingDataColumns.

Note: See TracTickets for help on using tickets.