DOC: Optimize: update DE documentation #17495
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
j-bowhay
added
scipy.optimize
Documentation
andyfaff
reviewed
Nov 30, 2022
|
|
||
|
|
||
| where the integers :math:`r1, r2, r3, r4 and r5` are chosen randomly | ||
| from the interval [1, NP] and are different from the |
There was a problem hiding this comment.
NP is defined a couple lines below. I think it's worth defining what NP is around about L80.
There was a problem hiding this comment.
not totally sure of the definition of NP.
Is it popsize*N as described in L80 or is it just the size of our target vector b'?
|
@andyfaff thanks for feedback will update these things now. will update PR when complete. |
j-bowhay
changed the title
j-bowhay
changed the title
|
The docs seem to render ok. I removed extra text from @mkiffer original commit because it was identical to text from a journal article, which would be under copyright. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reference issue
Closes #17398
What does this implement/fix?
Additional information
Documentation has been added regarding the use of evolution strategies.
I believe this update may need some assistance. I was not able to check my documentation fits the style requirements. I will submit this pull request and welcome any and all feedback.
I used the paper as posted by @andyfaff which had some fantastic information. However I am not totally clear on the differences between binomial and exponential implementation of DE optimisation.