Opened 8 years ago

Closed 8 years ago

#1571 closed defect (done)

Bug in the implementation of Polynomial*PositionManipulator

Reported by: abeham Owned by: abeham
Priority: medium Milestone: HeuristicLab 3.3.5
Component: Encodings.RealVectorEncoding Version: 3.3.5
Keywords: Cc:

Description

There is a bug in the implementation:

For the PolynomialOnePositionManipulator consider Line 90 -, two parentheses are lost in the expression:

Currently it is: delta = 1.0 - Math.Pow(2.0 - 2.0 * u, 1.0 / contiguity.Value + 1);

Should be: delta = 1.0 - Math.Pow(2.0 - 2.0 * u, 1.0 / (contiguity.Value + 1));

same for PolynomialAllPositionManipulator

Change History (4)

comment:1 Changed 8 years ago by abeham

  • Status changed from new to accepted

r6506

  • fixed bug in the Polynomial*PositionManipulator
  • updated test case

comment:2 Changed 8 years ago by abeham

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

comment:3 Changed 8 years ago by mkommend

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

comment:4 Changed 8 years ago by swagner

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