You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.
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.
The text was updated successfully, but these errors were encountered: