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

Expected improvement per second #412

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

ja-thomas
Copy link
Contributor

Allow infill crits access to the time.model

Implement expected improvement per second

ToDo:

  • Fix infillcrit printer
  • Tests

@ja-thomas
Copy link
Contributor Author

Also, I need to check the snoek paper if the time is modeled in log space or only the logtransform is only done on the predictions. I think the first one is correct but I did the latter...

@ja-thomas ja-thomas changed the title Expected improvement per second [not quite done] Expected improvement per second Feb 6, 2018
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

Successfully merging this pull request may close these issues.

1 participant