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
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
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: ↓ 14 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
comment:17 Changed 6 years ago by mkommend
- Resolution set to done
- Status changed from readytorelease to closed
My two cents: