-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
create convenient "entrypoints" for different MBO variants #138
Comments
For every REASONABLE MBO variant we write a an easy to use function. And before we do this, we should carefully write down the API, so we are SURE we really make stuff easier and dont overcomplicate things. |
Actually, it might be easier to simply have makeEGOControl? |
The reason behind a ego function is that we could also define the surrogate learner ourselves and increase simplicity. |
Both is just confusing for the user and too much too maintain. |
Did the same in ecr. I agree, that it is somehow a maintainence overload, but has some benefits as well especially in hiding the creation of the control object and offering a more "R-like" interface. |
The thing is: |
I should read the entire log here and not just the last few postings I guess 😄 |
I created a branch: You can compare here. |
See PR #234 |
for example call
ego(fun, ...)
and no more control objects are needed for simple stuff.The text was updated successfully, but these errors were encountered: