[jboss-jira] [JBoss JIRA] (JBRULES-3334) AbstractTabuAcceptor doesn't add the starting solution as tabu

[Geoffrey De Smet (Commented) (JIRA)]

    [ https://issues.jboss.org/browse/JBRULES-3334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12654470#comment-12654470 ] 

Geoffrey De Smet commented on JBRULES-3334:
-------------------------------------------

+1 to adding it in the phaseStarted() method, after calling super.phaseStarted()

Are you open to making a pull request (just for this change alone)?
Otherwise, I 'll quick fix it after my vehicle routing improvements are done.
Thanks for reporting!
                
> AbstractTabuAcceptor doesn't add the starting solution as tabu
> --------------------------------------------------------------
>
>                 Key: JBRULES-3334
>                 URL: https://issues.jboss.org/browse/JBRULES-3334
>             Project: Drools
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: drools-planner
>    Affects Versions: 5.3.1.Final
>            Reporter: Patrik Dufresne
>            Assignee: Geoffrey De Smet
>
> When setting a complete solution tabu, the starting solution is not considered a Tabu making it possible to visit the same solution again.
> Consider a situation where there is construction phase followed by a local search phase. The construction phase will create a solution that will not be added to the tabu list of the local search phased.
> To fix it, I suggest to override the function phaseStarted(...) in SolutionTabuAcceptor to add solution to the Tabu map.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira