Free cookie consent management tool by TermsFeed Policy Generator

Opened 7 years ago

Closed 6 years ago

#2890 closed enhancement (done)

Overhaul of GradientBoostedTreesModelView

Reported by: fholzing Owned by: mkommend
Priority: low Milestone: HeuristicLab 3.3.16
Component: Algorithms.DataAnalysis.Views Version: trunk
Keywords: Performance Cc:

Description (last modified by fholzing)

A performance problem occurs if the Parameters Iterations and/or Maximum Tree Size are chosen too big.

Reason: The listBox has runtime-troubles with longer strings (required amount of time grows with Maximum Tree Size times Iterations).

Note mkommend: Alternative solutions (ListView etc.) are preferred.

Additional task: the duplicate ViewHost.ViewsLabelVisible should be removed. Additional task: unification with Model evaluation view

Please note: r15756 still remains in the trunk, the following commits are in the branch

Change History (17)

comment:1 Changed 7 years ago by gkronber

My two cents:

  • max tree size in GBT should be limited. Usual values are 3 - 10. 20 nodes are too many.
  • it is not necessary to show the complete model string in the listbox. it would be sufficient to show only e.g. "Model 1" - "Model N".

comment:2 Changed 7 years ago by fholzing

r15730: Changed from ListBox to ListView; Added GroupBox to underline to distinction between outer and inner ViewHost; Simplified naming to "Model 1 - Model N"

comment:3 Changed 7 years ago by fholzing

  • Status changed from new to accepted

comment:4 Changed 7 years ago by fholzing

  • Owner changed from fholzing to mkommend
  • Status changed from accepted to reviewing

comment:5 Changed 7 years ago by mkommend

  • Owner changed from mkommend to fholzing
  • Status changed from reviewing to assigned

Please adapt the change the GBT Model evaluation view accordingly or unify the functionality of these two views.

comment:6 Changed 7 years ago by fholzing

r15756: Created branch

Note: r15730 still remains in the trunk

comment:7 Changed 7 years ago by fholzing

  • Description modified (diff)
  • Version changed from trunk to branch

comment:8 Changed 7 years ago by fholzing

r15759: Made the branch compile

comment:9 follow-up: Changed 7 years ago by fholzing

After consultation with mkommend the branch can be closed without any changes. We can't unify the two views.

comment:10 Changed 7 years ago by fholzing

  • Version changed from branch to trunk

comment:11 Changed 7 years ago by fholzing

  • Description modified (diff)

comment:12 Changed 7 years ago by fholzing

r15782: Aligned ListView with GroupBox and renamed GroupBox from Visualization to Representation

comment:13 Changed 7 years ago by fholzing

  • Owner changed from fholzing to mkommend
  • Status changed from assigned to reviewing

comment:14 in reply to: ↑ 9 Changed 7 years ago by fholzing

r15781: Removed branch

comment:15 Changed 7 years ago by mkommend

  • Status changed from reviewing to readytorelease

Reviewed r15782.

comment:16 Changed 6 years ago by mkommend

r15997: Merged r15730, r15782 into stable.

comment:17 Changed 6 years ago by mkommend

  • Resolution set to done
  • Status changed from readytorelease to closed
Note: See TracTickets for help on using tickets.