Free cookie consent management tool by TermsFeed Policy Generator

Opened 9 years ago

Closed 9 years ago

#2398 closed defect (done)

Pruning operators mistakenly try to calculate impacts for the program root symbol and the start symbol

Reported by: bburlacu Owned by: gkronber
Priority: medium Milestone: HeuristicLab 3.3.12
Component: Problems.DataAnalysis.Symbolic Version: 3.3.11
Keywords: Cc:

Description

Both regression and classification pruning operators try to calculate impacts for all the tree nodes, when they should be doing so only for the nodes below the start symbol. This leads to an exception by the interpreter when a program root symbol (or start symbol) is added as a child under the start symbol of the temp tree used for calculating replacement values.

Change History (9)

comment:1 Changed 9 years ago by bburlacu

  • Status changed from new to accepted
  • Summary changed from Pruning operators mistakingly try to calculate impacts for the program root symbol and the start symbol to Pruning operators mistakenly try to calculate impacts for the program root symbol and the start symbol

comment:2 Changed 9 years ago by bburlacu

r12461: Skip root and start symbols when calculating impacts and replacement values in the pruning operators.

comment:3 Changed 9 years ago by bburlacu

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

comment:4 Changed 9 years ago by ascheibe

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

comment:5 Changed 9 years ago by ascheibe

  • Status changed from assigned to reviewing

comment:6 Changed 9 years ago by gkronber

  • Status changed from reviewing to readytorelease

Reviewed r12461.

comment:7 Changed 9 years ago by gkronber

Depends on #2359 which needs to be merged first.

comment:8 Changed 9 years ago by gkronber

r12745: (combined stable merge #2359) merged r12189, r12358, r12359, r12361, r12461, r12674, r12720, r12744 from trunk to stable

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

comment:9 Changed 9 years ago by gkronber

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