Islands Optimizer requires Lexicase
JorySchossau opened this issue · 3 comments
The Optimizers must all be present in buildOptions for a successful build.
Should be documented (and more clearly named) if this is the correct behavior.
I'm not sure how best to address this.
Because of MABEs absolutely crap include process (modules.h, etc.) we must include any optimizer that needs to be used in an island directly. So, currently, yes, you need lexicase and simple optimizer to be in the build if you want to use island optimizer. I'm not sure where to document it. If we add it to the code, it does not help anyone who does not know how to read the code (and if you can read the code it's very clear from the constructor what is happening). If you are not going to look at the code, where should this be documented?
I was thinking we could change the name from Islands to IslandsAddon_Simple_Lexicase or something like that. Yeah it's ugly, but we don't exactly go about typing the name of the optimizer hardly ever.
I'm not sure what to do here.
On the one hand, we could move this to extras. But it's the only instance that uses namespaced parameters anywhere in the stuff that ships with core.