[
https://issues.apache.org/jira/browse/MATH-207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12719319#action_12719319
]
Phil Steitz commented on MATH-207:
----------------------------------
Committed a slightly modified version of David's last patch in r784604.
Other than minor javadoc/formatting changes to make checkstyle happy, I also
made the shared source of randomness pluggable.
I am not 100% happy with the static randomGenerator attached to
GeneticAlgorithm, though I understand and support the need to ensure
reproducibility for some applications. Comments / suggestions for better ways
to do this welcome.
Leaving open as we need to update the user guide to complete this.
> Implementation of GeneticAlgorithm.nextGeneration() is wrong
> ------------------------------------------------------------
>
> Key: MATH-207
> URL: https://issues.apache.org/jira/browse/MATH-207
> Project: Commons Math
> Issue Type: Bug
> Affects Versions: 2.0
> Reporter: David Stefka
> Assignee: Phil Steitz
> Fix For: 2.0
>
> Attachments: geneticalgorithm.patch, geneticAlgorithms.zip,
> genetics_impl.zip, patch
>
> Original Estimate: 0.17h
> Remaining Estimate: 0.17h
>
> The implementation of GeneticAlgorithm.nextGeneration() is wrong, since the
> only way how a Chromosome can get into the new generation is by mutation.
> Enclosed, I am sending a patch for this.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
