-
Notifications
You must be signed in to change notification settings - Fork 161
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
nsga2 batch fitness evaluation (bfe) #308
Conversation
Thanks for this PR, can you fix the CI failures? Mainly related to dangerous conversions among int types? |
Yes, I'll be glad to sort these out. Thanks for drawing my attention to them as I haddn't noticed.
…On 2019-06-15 22:12, Dario Izzo wrote:
Thanks for this PR, can you fix the CI failures? Mainly related to
dangerous conversions among int types?
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the
thread [2].
Links:
------
[1]
#308?email_source=notifications&email_token=AISL5WTHACTXICFUNEFSAJ3P2VLNFA5CNFSM4HYPNPZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXZAGLA#issuecomment-502399788
[2]
https://github.com/notifications/unsubscribe-auth/AISL5WW3LMMS5R22KG3EXOTP2VLNFANCNFSM4HYPNPZA
|
All done. Sorry I overlooked them before. |
src/algorithms/nsga2.cpp
Outdated
|
||
poptemp.push_back(child1); | ||
poptemp.push_back(child2); | ||
//popnew.push_back(child1, f1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@willcodeforfudd can these commented lines be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, of course. Sorry for the sloppy code, and repeated commits - this is my first time collaborating on a github project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@willcodeforfudd no worries, thanks for contributing the code. I am trying to ping @darioizzo to see if he can give the final OK for merging.
@willcodeforfudd thanks for the PR! I had a small comment, but otherwise it looks good to me 👍 |
I think that these are the minimal changes to get working with bfe on nsga2