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

split perform_uh in 3 #139

Merged
merged 3 commits into from Jan 12, 2016
Merged

split perform_uh in 3 #139

merged 3 commits into from Jan 12, 2016

Conversation

montanier
Copy link
Contributor

I have an evaluation scheme that does not fit simply in one function. As a consequence, it was pretty difficult to realize the reevaluation of the solutions. I have split perform_uh in three:

  • select the candidates to reevalute
  • reevaluate the candidates
  • set the new values to the solutions.

This allow me to use my own tools to perform the reevaluation and give the results back to the libcmaes. I have verify that my modification does not brake perform_uh by using the test_functions utility.

@beniz
Copy link
Collaborator

beniz commented Jan 8, 2016

Thanks, looks good to me: more flexibility does not hurt.

If you have an example to share, and possibly the underlying reasons that make an evaluation scheme interleave with the uncertainty handling procedure, it would be interesting I think.

@montanier
Copy link
Contributor Author

True, that would help to understand and show the possibilities of the lib.
Currently my code using the libcmaes requires too many external elements to
be a good example. I will add that as soon as I have the time to do
something small and clear.

On Fri, Jan 8, 2016 at 5:23 PM, Emmanuel Benazera notifications@github.com
wrote:

Thanks, looks good to me: more flexibility does not hurt.

If you have an example to share, and possibly the underlying reasons that
make an evaluation scheme interleave with the uncertainty handling
procedure, it would be interesting I think.


Reply to this email directly or view it on GitHub
#139 (comment).

beniz added a commit that referenced this pull request Jan 12, 2016
split perform_uh in 3 sub-procedures
@beniz beniz merged commit 0290d4f into CMA-ES:master Jan 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants