Opened 8 years ago
Last modified 7 years ago
#2709 closed enhancement
DataPreprocessing Enhancements — at Version 37
Reported by: | pfleck | Owned by: | pfleck |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.15 |
Component: | DataPreprocessing.Views | Version: | 3.3.14 |
Keywords: | Cc: |
Description (last modified by pfleck)
This ticket contains smaller visual enhancements of the preprocessing views.
This ticket depends on #2713.
- moved DataTable/ScatterPlotControl out of DataTable/ScatterPlotView
- introduced regression curves in scatterplot
This ticket depends on #2715.
- introduce Histogram aggregation
Enhancements
- ViewHost/ViewShortcut usage
Remove ViewHost icons for the ViewShortcutsSplit Single- and Multi-Scatterplot
Remove “View Shortcuts” grouping box- Doubleclick a ViewShortcut should not reset the state in the new view
PreprocessingCheckedItemViewHide Move, Add, Delete buttonsAdd Select Input/Target, All and None as Buttons (checkboxes with tooltips)Remove context menu instead
DataGrid + StatisticsShow/Hide columns and rowsCheck All, Input/Target, None Variables optionInitially only Input/Target variables should be checked
DataCompletnessChartRemove titleMove legend to top (column style)
ScatterplotBetter default axis ranges (Bogdans helper functions)Axis description instead of legendManual axis-range (also for linechart)(currently via config dialog)
MultiScatterplotX-axis labels vertical
New features
Distinguish Color and Grouping option in scatterplotCurrent “Color” feature becomes “Grouping”“Color” should be possible for all features, using the Color Gradient
ScatterplotAdd slider for changing point size(currently via config dialog)Add regression line and add option to show/hide(implemented in #2713)
MultiScatterplotAdd (better) tooltips(are legend tooltip in #2713)Add correlation coefficient to scatterplot(visible in tooltip of legend)
Histogram + MultiLinechartAdd chart size sliders (as in MultiScatterplot)or column count?
- Feature Correlation Matrix
- Check All, Input/Target, None Variables option
- New Button should open a “are you sure current data is deleted” dialog
Change History (37)
comment:1 Changed 8 years ago by mkommend
- Summary changed from Preprocessing Visual Enhancements to DataPreprocessing Enhancements
comment:2 Changed 8 years ago by pfleck
- Description modified (diff)
comment:3 Changed 8 years ago by pfleck
- Status changed from new to accepted
comment:4 Changed 8 years ago by pfleck
comment:5 Changed 8 years ago by pfleck
r14445 Branched DataPreprocessing plugins. Adapted build paths and references.
comment:6 Changed 8 years ago by pfleck
r14446 Removed the PreprocessingScatterPlotView and use the HL ScatterPlotControl instead.
comment:7 Changed 8 years ago by pfleck
- Removed the PreprocessingDataTable and PreprocessingDataTableView and use dhe HL DatatTableControl instead.
- Moved and refactored some code of PreprocessingChart and moved unnecessary code from base classes to actual derivative classes.
Some features of the PreprocessingDataTableView are included in the regular DataTableView in #2715.
comment:8 Changed 8 years ago by pfleck
- Description modified (diff)
comment:9 Changed 8 years ago by pfleck
r14460 Fixed missing resx in csproj.
comment:10 Changed 8 years ago by pfleck
- Description modified (diff)
- Added a separate MultiScatterPlot entry and removed the ViewHost views-icon instead.
- Moved legend of DataCompletenessChart to the top and removed the title instead.
comment:11 Changed 8 years ago by pfleck
- Description modified (diff)
- Removed some groupboxes in ViewShortcutListView.
- Removed unnecessary IViewChartShortcut
- Split ScatterPlot Multi and Single in to separate contents.
- Renamed Color-combo box in Scatterplot to "Group".
comment:12 Changed 8 years ago by pfleck
- Fixed bugs with double-click on view shortcut.
- Reuse visual properties for single scatterplot.
comment:13 Changed 8 years ago by pfleck
- Description modified (diff)
r14472 Better initial axis intervals for scatterplots.
comment:14 Changed 8 years ago by pfleck
r14473 Improved default y-axis for line charts.
comment:15 Changed 8 years ago by pfleck
- Improved legend description for grouped histogram and scatterplots.
- Fixed initial size of points for scatterplots.
- Added correlation calculation for scatterplots (not used yet).
comment:16 Changed 8 years ago by pfleck
- Description modified (diff)
- Fixed initial point size for scatterplots.
- Reuse the visual properties of the old data row if a single variable is changed in the ScatterPlotSingleView
comment:17 Changed 8 years ago by pfleck
- Description modified (diff)
- Added Check Inputs/All/None buttons instead of showing disabled buttons of the ItemCollectionView.
- Removed the PreprocessingCheckedItemListView. A standard ListView is used instead.
- Fixed slow updating when simultaneously (un-)checking multiple variables in the chart views. (currently only works by using the new buttons)
comment:18 Changed 8 years ago by pfleck
- Description modified (diff)
- Added a VerticalLabel for the multi-scatterplot.
- Added regression options for single- and multi-scatterplot
comment:19 Changed 8 years ago by pfleck
- Description modified (diff)
r14512 Added an option for the preprocessing scatterplot to use a color gradient instead of the chart color palette.
comment:20 Changed 8 years ago by pfleck
- Added suggestion feature for singlescatterplotview.
- Shows NaN groups in scatterplot (black if gradient is selected).
- Only enables input variables in DataGridContentView per default.
- Added missing resx file (gradient image).
comment:21 Changed 8 years ago by pfleck
- Description modified (diff)
- Uses StringMatrix for statistics instead of winforms datagrid.
- Precheck input/target variables only for statistics.
comment:22 Changed 8 years ago by pfleck
- Description modified (diff)
r14546 Added shortcuts for select input/all/none variables in datagrid and statistics.
comment:23 Changed 8 years ago by pfleck
- Owner changed from pfleck to mkommend
- Status changed from accepted to reviewing
comment:24 Changed 8 years ago by pfleck
r14578 Fixed wrongly positioned options in histogram view.
comment:25 Changed 8 years ago by mkommend
r14579: Refactored histogram view and content to support grouping by string and datetime variables.
comment:26 Changed 8 years ago by mkommend
r14580: Changed initialization of caches to avoid NullReferenceExceptions.
comment:27 Changed 8 years ago by mkommend
r14581: Refactored get variables for grouping (extracted method to another class).
comment:28 Changed 8 years ago by pfleck
- Added histogram aggregation option.
- Show all columns in data grid per default.
comment:29 Changed 8 years ago by mkommend
r14723: Updated branch with most recent trunk changes.
comment:30 Changed 8 years ago by mkommend
Testing
View shortcuts should have more descriptive names and use spaces instead of camel case, for example "Line chart" instead of "LineChart".- All multi XXX chart should support opening an individual chart in a new tab by double clicking them
Data gridWhat is the point of showing no variables? Especialle because the show column context menu cannot be opened anymore.Spacing between row / column count & action button should be the same as for action buttons & the show variablesShow Variables GroupBox just as label or centered. Currently it looks slightly odd.
- Statistics
Horizontally listed columns look much better.However, would it be possible to configure the direction (horizontally vs vertically)Show Variables GroupBox should be layouted vertically to use the available space better.The datagrid shows per default all columns, whereas statistics only show the inputs + target. Per default all variables should be shown in the data grid and statistics view,but non-inputs should be highlighted maybe italic.
- Line chart
- Check and uncheck all variables have unintuitive icons. Can't you use a checked and unchecked box? (Applies to the histogram as well).
- Reuse the icons for the data grid and statistics as well?
Size / Column count slider is missing. (Applies to the histogram as well).
HistogramTitle font is increased when enable grouping.Aggregation options are pretty cool.There should be an option to order the legend alphabetically instead of based on the occurance in the data(comment:47).
Scatter plotIt should be possible to change the point size and transparency of the data points. (Applies to the multi scatter plot as well).(comment:45)More reasonable default text size.
Multi Scatter plotIt should only have one size slider instead of two separate ones for width and height
Review Comments
Chart classes should be sealed and members should be private (e.g. LineChartView).(will be done in a separate ticket on general DataPreprocessing architecture overhaul)Commented code should be removed (PreprocessingChartView).(comment:50)Remove resx files (ScatterPlotSingleView)(resx in ScatterPlotSingleView contains the gradient image)
comment:31 Changed 8 years ago by mkommend
- Status changed from reviewing to assigned
comment:32 Changed 8 years ago by mkommend
- Status changed from assigned to accepted
comment:33 Changed 8 years ago by mkommend
r14724: Adapted data preprocessing scatter plot to allow grouping of string variables.
comment:34 Changed 8 years ago by mkommend
- Owner changed from mkommend to pfleck
- Status changed from accepted to assigned
r14725: Added grouping for multi scatter plot view.
comment:35 Changed 8 years ago by pfleck
- Status changed from assigned to accepted
comment:36 Changed 8 years ago by pfleck
- Description modified (diff)
- Changed chart sizing to absolute values (pixels).
- Added chart sizing to Linechart and Histogram.
comment:37 Changed 8 years ago by pfleck
- Description modified (diff)
r14440 created branch
r14441 Copied plugins.