Remove ParallelStrategies from Producer and make them reusable modules for algorithms instead. #760
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
They will become part of algorithms instead of being held outside of them. Algorithms need to handle non completed trials differently.
The parallel strategy will now be part of algorithm configuration. There is nothing to configure in the producer so it is removed altogether from experiment configuration. Any configuration passed for parallel strategy will trigger a warning and be ignored. It would be too complicated to try to map this to algorithm arguments for parallel strategies. Deprecation is planned for v0.4.
Some algorithms may require different parallel strategy for trials of different status. For instance, a pending trial has a better potential than a broken trial which should be assigned a very bad objective. The StatusBasedParallelStrategy can be configured to apply different strategies based on the trial status.
bouthilx
added
the
enhancement
Delaunay
approved these changes
Jan 11, 2022
Merged
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.
Fixes #602