Skip to content
Snippets Groups Projects
Commit ebe115c1 authored by kohlhaasrebecca's avatar kohlhaasrebecca
Browse files

Removed the sequential functions from the Engine class

parent f18ea5aa
No related branches found
No related tags found
2 merge requests!32Merge last changes for new release,!30Moved AL into SeqDesign and restructured score combination
...@@ -234,18 +234,28 @@ if __name__ == "__main__": ...@@ -234,18 +234,28 @@ if __name__ == "__main__":
This part of the code tests the available combinations of the This part of the code tests the available combinations of the
sequential strategies. sequential strategies.
The following combinations have remaining issues to be solved: The following combinations have remaining issues to be solved:
- exploitation = 'BayesOptDesign' has circular dependency with the engine class
- exploration = 'Voronoi' creates mismatching numbers of candidates and weights - exploration = 'Voronoi' creates mismatching numbers of candidates and weights
- exploration = 'loocv' needs MetaModel.create_ModelError, which does not exist - exploration = 'loocv' needs MetaModel.create_ModelError, which does not exist
- exploration = 'dual annealing' restricted in what it allows as exploitation methods - exploration = 'dual annealing' restricted in what it allows as exploitation methods
- tradeoff = 'adaptive' perhaps not possible in the first AL iteration? - tradeoff = 'adaptive' perhaps not possible in the first AL iteration?
The following combinations are running through: The following combinations are running through:
- BayesActDesign quite fast Tradeoff
- VarOptDesign as well - None
- equal
- epsilon-decreasing
Exploration
- random
- global_mc
Exploitation
- BayesActDesign
- VarOptDesign
- alphabetic
- space-filling
Performance notes: Performance notes:
- BayesOptDesign slow in the OptBayesianDesign iterations - BayesOptDesign slow in the OptBayesianDesign iterations
- exploitation = 'BayesOptDesign' has circular dependency with the engine class
# TODO: user-defined options were left out in this test for both # TODO: user-defined options were left out in this test for both
sampling-methods and exploitation schemes. sampling-methods and exploitation schemes.
......
This diff is collapsed.
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment