Free cookie consent management tool by TermsFeed Policy Generator

Opened 8 years ago

Closed 8 years ago

#2566 closed enhancement (done)

Improve collecting (Before|After)ExecutionOperators in Run

Reported by: abeham Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.14
Component: Operators Version: 3.3.13
Keywords: Cc:

Description

In a default configuration of a genetic algorithm and a tsp there are following parameters in the run:

  1. Analyzer.AfterExecutionOperators
  2. Analyzer.BeforeExecutionOperators
  3. Crossover.AfterExecutionOperators
  4. ...

In total there are 12 entries that all contain an empty OperatorList which unnecessarily bloats the run information. On the other hand, the operators within the OperatorList are _not_ collected and do not appear in the run, except as an instance inside the OperatorList.

I would recommend to not include the OperatorList itself by default (initialize ShowInRun to false for InstrumentedOperator) and instead collect run information from any contained opeartor.

Change History (6)

comment:1 Changed 8 years ago by abeham

  • Owner set to abeham
  • Status changed from new to accepted

comment:2 Changed 8 years ago by abeham

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

r13566: improved collection of parameter values in InstrumentedOperator

comment:3 Changed 8 years ago by mkommend

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

Reviewed and tested r13566.

The only comment I have is that in line 86/87 a hard cast should be used or an exception thrown if the value is not an OperatorList. The reason behind this is, that if it fails silently (as it is the case now), a breaking change will not be detected.

comment:4 Changed 8 years ago by abeham

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

r14103: implemented hard cast

comment:5 Changed 8 years ago by mkommend

  • Status changed from reviewing to readytorelease

comment:6 Changed 8 years ago by mkommend

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

r14114: Merged r13566, r14103 into stable.

Note: See TracTickets for help on using tickets.