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

Benchmark functions were locked into one of the tests #29

Merged
merged 2 commits into from Mar 29, 2016

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

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.
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.

None yet

2 participants