Opened 2 months ago

Closed 2 months ago

#2807 closed defect (done)

PartialDependencePlot throws Exception with constant values

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

Description

Could be easily tested with a LR and Miba CF3.

Change History (7)

comment:1 Changed 2 months ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 2 months ago by mkommend

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

r15211: Made partial dependence plot work with constant variable values.

comment:3 Changed 2 months ago by mkommend

During implementation of r15211 I noticed several cross-thread access exception (GUI components) in the FactorPartialDependencePlots.

comment:4 Changed 2 months ago by mkommend

r15213: Initialized trainingMin and trainingMax with reasonable values, so that trainingMin is actually smaller than trainingMax.

comment:5 Changed 2 months ago by abeham

  • Owner changed from abeham to mkommend
  • Status changed from reviewing to readytorelease

Tested it and it works. One cannot draw the slider however as in charts where there is not just one value. I don't know if this is intentional and I don't think this is very important.

Technically, the comment at line 257 resp. 353 is not valid anymore.

comment:6 Changed 2 months ago by mkommend

r15222: Removed obsolete comments from partial dependence plots.

Thanks for spotting this.

comment:7 Changed 2 months ago by mkommend

  • Resolution set to done
  • Status changed from readytorelease to closed

r15239: Merged r15211, r15213, r15222 into stable.

Note: See TracTickets for help on using tickets.