Free cookie consent management tool by TermsFeed Policy Generator

Opened 13 years ago

Closed 13 years ago

#1346 closed defect (done)

There is a problem serializing some VRP move operators

Reported by: svonolfe Owned by: svonolfe
Priority: high Milestone: HeuristicLab 3.3.3
Component: Problems.VehicleRouting Version: 3.3.3
Keywords: Cc:

Description

Some VRP move operators cannot be properly serialized / deserialized

Change History (11)

comment:1 Changed 13 years ago by svonolfe

Fixed serialization of some VRP move operators in trunk in r5129

comment:2 Changed 13 years ago by svonolfe

Fixed serialization of some VRP move operators in branch in r5130

comment:3 Changed 13 years ago by svonolfe

  • Status changed from new to accepted

comment:4 Changed 13 years ago by svonolfe

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

comment:5 Changed 13 years ago by abeham

  • Owner changed from abeham to svonolfe

r5347

  • Fixed cloning behavior in some operators
  • Fixed problem UI textbox length

Please review the changes made to AlbaIntraRouteInversionMove:

  • The permutation is cloned in the base operator already
  • What is the reason of making a clone of the permutation in the constructor?

comment:6 Changed 13 years ago by abeham

  • Version changed from 3.3.3 to 3.3.2

corrected version

comment:7 Changed 13 years ago by svonolfe

  • Status changed from reviewing to readytorelease

comment:8 Changed 13 years ago by abeham

  • Status changed from readytorelease to reviewing

Please provide a short comment along with putting it ready-to-release.

comment:9 Changed 13 years ago by svonolfe

  • Status changed from reviewing to readytorelease

The reason why a clone is created in the constructor is related to ticket #1185. I am afraid we have to leave it that way until that issue is fixed, otherwise the move cannot be applied correctly.

The other change is ok as well, the permutation is cloned in the base class already (TwoIndexMove).

PS: Greetings Andreas :)

comment:10 Changed 13 years ago by abeham

Okay, thanks for the reminder.

comment:11 Changed 13 years ago by mkommend

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