Free cookie consent management tool by TermsFeed Policy Generator

Opened 7 years ago

Last modified 6 years ago

#2695 accepted enhancement

Line charts for regression should show an ID column along the x-axis

Reported by: gkronber Owned by: gkronber
Priority: high Milestone: HeuristicLab 4.x Backlog
Component: Problems.DataAnalysis.Views Version: branch
Keywords: Cc:

Description (last modified by gkronber)

For instance if the data are ordered by time the time should be shown on the x-axis.

Change History (13)

comment:1 Changed 7 years ago by mkommend

  • Description modified (diff)
  • Summary changed from Line charts for regression should show an ID column along the y-axis to Line charts for regression should show an ID column along the x-axis

comment:2 Changed 7 years ago by gkronber

  • Milestone changed from HeuristicLab 3.3.15 to HeuristicLab 3.3.x Backlog

comment:3 Changed 7 years ago by gkronber

  • Milestone changed from HeuristicLab 3.3.x Backlog to HeuristicLab 3.3.16
  • Owner set to gkronber
  • Status changed from new to accepted

comment:4 Changed 7 years ago by gkronber

  • Version changed from 3.3.14 to branch

comment:5 Changed 7 years ago by gkronber

r15296: created a feature branch from the trunk

comment:6 Changed 7 years ago by gkronber

r15297: added a parameter to DataAnalysisProblemData to select an id column and extended RegressionLinechartViewBase to show the id on the xAxis

comment:7 Changed 7 years ago by gkronber

r15298: removed commented code and unused usings

comment:8 Changed 7 years ago by mkommend

Comments:

  • The branch does not compile on my machine (Errors in simulated annealing)
  • -> Could not test the implemented feature
  • IDataAnalysisProblemData & DataAnalysisProblemData
    • IMO IdVariable looks and sounds strange. What about IdentificationVariable, Identification, LabelVariable, ....
    • Why not programmatic setter for the IdVariable? I know it is cumbersome to set constrained value parameters by code, but it is even more so when no setter property is provided.
    • AllIds, TrainingIds, and TestIds could be unified by using a new method GetIds(IEnumerable<int> rows)
    • Why are only features with unique values for every row allowed?
    • Ids should only be reset if the dataset changes on not if the partitions are updated
    • Is it possible to change the ID after solution creation (e.g. for updating the labels in a line chart) or is this not considered?

Code in LineChartBase looks reasonable. However, all other views should be updated as well (EstimatedValuesView).

comment:9 Changed 7 years ago by gkronber

r15310: merged r15299:15302 from trunk to branch

comment:10 Changed 6 years ago by gkronber

r15648: renamed folder for branch

comment:11 Changed 6 years ago by gkronber

r15648 renamed folder for branch

comment:12 Changed 6 years ago by gkronber

  • Description modified (diff)
  • Milestone changed from HeuristicLab 3.3.16 to HeuristicLab 4.x Backlog

comment:13 Changed 6 years ago by gkronber

  • Component changed from Problems.DataAnalysis.Symbolic.Regression.Views to Problems.DataAnalysis.Views
  • Priority changed from medium to high
Note: See TracTickets for help on using tickets.