-
Notifications
You must be signed in to change notification settings - Fork 152
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
[WIP] DDS new algorithm #185
Closed
Closed
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
…ectivefunction* support still open.
* master: Removed _algorithm.model, since it is not legal to call. Use _algorithm.simulate instead Removed all calls to self.model in non-funct samplers and replaced it with calls to _algorithm.simulate Accept List Parameter only for MC sampler Support for mixed paramter generation reactivated. List are not properly tested for different algos Removed unittests for mixed parameter generation. Drop of support Enable dealing with non class parameters Add missing default tuple Eanble the filtering of Constant Parameters in _algorithm.py Changed order of parameters to ensure test on Python < 3.6 Fixed thouska#175 and thouska#166 with extensive check to preserve the constants in sceua and developed a number of tests to ensure the constants stays constant in all samplers. Fixed thouska#175 and thouska#166 with extensive check to preserve the constants in sceua and developed a number of tests to ensure the constants stays constant in all samplers. Added tests for named parameters for all samplers. Fails for mle, sa and sceua Fixed Constant parameter usage for sceua (thouska#166 and thouska#175) and build test cases Added a mechanism to exclude parameter types from a sampler, especially List parameters from any sampler except mc and list Change default dbformat to ram
Pull Request Test Coverage Report for Build 601
💛 - Coveralls |
if i_left <= 0: | ||
raise ValueError('# Initialization samples >= Max # function evaluations.') | ||
|
||
starting_generator = ( |
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.
Use self.parameter() distribution function. Maybe we need to write a parameter.Discrete distribution
I messed up a commit, so I started a new PR, see: #195. |
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.
Pull request, first version, for discuss. Please do not merge immediately.