8000 Benchmark functions were locked into one of the tests by betatim · Pull Request #29 · scikit-optimize/scikit-optimize · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Feb 28, 2024. It is now read-only.

Conversation

betatim
Copy link
Member
@betatim betatim commented Mar 29, 2016

Refactored benchmark functions into their own module and added
some basic (in)sanity tests for each.

[3, 3.5, 1.7, 10, 17, 8],
[17, 8, 0.05, 10, 0.1, 14]])):
return -np.sum(alpha * np.exp(-np.sum(A * (x - P)**2, axis=1)))
from skopt.benchmarks import branin, hartmann_6
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one import per line

@glouppe
Copy link
Member
glouppe commented Mar 29, 2016

Thanks, much better this way! +1 after the small changes on imports

@betatim
Copy link
Member Author
8000
betatim commented Mar 29, 2016

This adds some of the functions mentioned in #16 unsure if we should close it or leave it open to attract more benchmark problems.

@glouppe glouppe merged commit adba1e0 into scikit-optimize:master Mar 29, 2016
@betatim
Copy link
Member Author
betatim commented Mar 29, 2016

Reverted this in #30, will try again in a new PR.

@betatim betatim deleted the benchmarks branch March 29, 2016 11:15
holgern added a commit that referenced this pull request Feb 11, 2020
Added checking-functions for readability and reusability.
< 5F1C form class="d-none js-timeline-marker-form" data-turbo="false" action="/_graphql/MarkNotificationSubjectAsRead" accept-charset="UTF-8" data-remote="true" method="post">
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0