Skip to content
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

Suggestions in fabolas.py #1

Open
cjfcsjt opened this issue Apr 29, 2019 · 1 comment
Open

Suggestions in fabolas.py #1

cjfcsjt opened this issue Apr 29, 2019 · 1 comment

Comments

@cjfcsjt
Copy link

cjfcsjt commented Apr 29, 2019

Hi, I’m a student and learning about BayesianOptimization rencently. I’m trying to make fabolas compactible to George 0.3.1 and I think I did it. And I hope I can give you some suggestions:

  1. I suggest that using stationary kernel (E.g. SE kernel) instead of non-stationary kernel (LinearKernel in Fabolas.py), because when you run get_incumbent() (in Fabolas.py), you will project the environment variables to 1, and then change to 0 because of _quadratic_bf(). Then you will run predict() in get_incumbent(), and the parameters of predict() will be matrix with env=0 (E.g. (a1,b1,0) ,(a2,b2,0) ,(a3,b3,0)...)
    If you use LinearKernel, the var of predict() will be a zeroes, and mean will of predict() will be a vector with same elements.As a result, the epmgp.py cannot work

  2. The parameter of EnvPrior() “n_lr=degree+1” can change to “n_lr= len(env_kernel) “

@RLeenings
Copy link
Collaborator

Thanks a lot. Did you manage to use Fabolas with your suggestions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants