[
https://issues.apache.org/jira/browse/MATH-207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12705721#action_12705721
]
Ben McCann commented on MATH-207:
---------------------------------
Sorry for my delayed response. JIRA didn't email me on any of these updates
for whatever reason.
There's no real need to make the fields protected in my patch. Private would
be fine. I've just gotten in the habit of frequently using protected to allow
easier subclassing and unit testing, but those fields all have public getters,
so no harm in making them private.
> 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.
