Replace get_random_elite() with batched sample_elites() method #192
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.
Description
This PR replaces
get_random_elite
with asample_elites
method which samples an entire batch of elites from the archive rather than just a single elite. This allows the emitter to operate more quickly since they do not have to callget_random_elite
one at a time. We have decided to rename the method so that we are guaranteed to break people who still use get_random_elite().TODO
sample_elites
get_random_elite
benchmarks (sincesample_elites
is batched, it is much faster)sample_elites
Questions
Status
yapf
pytest
pylint
HISTORY.md
.