Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pop::size is actually pop::size*2 #12

Open
jbmouret opened this issue Jul 21, 2016 · 6 comments
Open

pop::size is actually pop::size*2 #12

jbmouret opened this issue Jul 21, 2016 · 6 comments
Labels

Comments

@jbmouret
Copy link
Member

This does not change anything but this can be counter-intuitive when we evaluate the number of fitness evaluation.

OK to fix?

@jbmouret jbmouret added the bug label Jul 21, 2016
@costashatz
Copy link
Collaborator

Why is it pop::size*2?

@vassilisvas
Copy link
Collaborator

vassilisvas commented Jul 22, 2016

JB I am guessing that you are referring to the fact that two offspring are generated for each element in the population, this is why it is pop::size*2.

I am ok with fixing this.

@costashatz
Copy link
Collaborator

JB I am guessing that you are referring to the fact that two offspring are generated for each element in the population, this is why it is pop::size*2.

Ohh okay. That's true.

@jbmouret what fix are you suggesting?

@jbmouret
Copy link
Member Author

I propose to simply divide pop::size by 2 in the reproduction loop.

@vassilisvas
Copy link
Collaborator

I don't have a problem with this (or any) solution.

@costashatz
Copy link
Collaborator

I propose to simply divide pop::size by 2 in the reproduction loop.

Seems okay, but we need to double our pop::size in our experiments (in order to get the results of the same quality)..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants