Context Navigation

#2690 closed feature request (done)

Implement views for decision tree models (random forest and gradient boosted trees)

Reported by:	gkronber	Owned by:	gkronber
Priority:	medium	Milestone:	HeuristicLab 3.3.15
Component:	Algorithms.DataAnalysis.Views	Version:	3.3.14
Keywords:		Cc:

Description

It would be nice if we could inspect decision tree models

Change History (21)

comment:1 Changed 7 years ago by gkronber

Status changed from new to accepted

comment:2 Changed 7 years ago by gkronber

r14345: implemented methods to generate symbolic expression tree solutions for decision tree models (random forest and gradient boosted) as well as views which make it possible to inspect each of the individual trees in a GBT and RF solution

comment:3 Changed 7 years ago by gkronber

Owner changed from gkronber to mkommend
Status changed from accepted to reviewing

comment:4 Changed 7 years ago by gkronber

r14346: removed unnecessary plugin dependency

Last edited 7 years ago by gkronber (previous) (diff)

comment:5 follow-up: ↓ 6 Changed 7 years ago by mkommend

Owner changed from mkommend to gkronber
Status changed from reviewing to assigned

Bug: At least RF models are not stored directly, but rather recreated when necessary. However the new model view does not trigger this recreation and throws an out-of-bounds exception if a deserialized model is displayed.

comment:6 in reply to: ↑ 5 Changed 7 years ago by gkronber

Replying to mkommend:

Bug: At least RF models are not stored directly, but rather recreated when necessary. However the new model view does not trigger this recreation and throws an out-of-bounds exception if a deserialized model is displayed.

Thanks. Should be fixed with r14368.

comment:7 Changed 7 years ago by gkronber

Owner changed from gkronber to mkommend
Status changed from assigned to reviewing

comment:8 follow-up: ↓ 13 Changed 7 years ago by mkommend

First review comments during testing:

Double click in the list view does not open the symbolic regression solution.
Excel format for the symbolic models does not work correctly. There is no infinity symbol in excel and additionally 1/(1+EXP(-Infinity * ($D1-44)) is not easy to interpret correctly.

Version 0, edited 7 years ago by mkommend (next)

comment:9 Changed 7 years ago by mkommend

Owner changed from mkommend to gkronber
Status changed from reviewing to assigned

comment:10 Changed 7 years ago by mkommend

r14839: Adapated content attribute of solution views for GBT and RF.

This change has been necessary to show the solution name text box from the NamedDataAnalysisSolutionView.

comment:11 Changed 7 years ago by gkronber

r15103: added event-handler for double click events to open the clicked model in a new view

comment:12 Changed 7 years ago by gkronber

r15104: changed Excel-formatter for VariableCondition symbols to produce a correct output also for GBT and RF models

comment:13 in reply to: ↑ 8 Changed 7 years ago by gkronber

Replying to mkommend:

Review comments:

The model view is the last entry in the solution view.
Why are evaluation views misused to display the individual trees. Would not it be possible to create a view for IRandomForestModel / GBTModel?

This is not possible as the views for the individual trees need the problem data to calculate the results for the indiviual trees. However it would be possible to provide both views. Whereas, the model view only shows the trees and the other view is renamed.

Why are GBTSolutionView and RF Regression/Classifation View necessary? Only to remove the menu entry?

Yes, but probably not anymore if we keep both entries.

comment:14 Changed 7 years ago by gkronber

r15105:15106: added a model view for GBT models and renamed the existing evaluation view accordingly. Deleted GBT solution view because it is not necessary anymore.

Last edited 7 years ago by gkronber (previous) (diff)