Opened 10 years ago
Closed 10 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:
- Analyzer.AfterExecutionOperators
- Analyzer.BeforeExecutionOperators
- Crossover.AfterExecutionOperators
- ...
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 10 years ago by abeham
- Owner set to abeham
- Status changed from new to accepted
comment:2 Changed 10 years ago by abeham
- Owner changed from abeham to mkommend
- Status changed from accepted to reviewing
comment:3 Changed 10 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 10 years ago by abeham
- Owner changed from abeham to mkommend
- Status changed from assigned to reviewing
r14103: implemented hard cast
comment:5 Changed 10 years ago by mkommend
- Status changed from reviewing to readytorelease
comment:6 Changed 10 years ago by mkommend
- Resolution set to done
- Status changed from readytorelease to closed
r13566: improved collection of parameter values in InstrumentedOperator