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

Adding back restart functionality #34

Merged
merged 13 commits into from
Jul 5, 2019
Merged

Adding back restart functionality #34

merged 13 commits into from
Jul 5, 2019

Conversation

dme65
Copy link
Owner

@dme65 dme65 commented Jul 5, 2019

Adding an argument use_restarts(default True) to the optimization strategies. This starts a fresh optimization run if the budget is not exhausted and the optimization run has stopped making progress. We don't terminate pending evaluations when we restart, but we don't incorporate them into the surrogate model when they are completed.

@codecov
Copy link

codecov bot commented Jul 5, 2019

Codecov Report

Merging #34 into master will increase coverage by 0.31%.
The diff coverage is 90.24%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   92.53%   92.84%   +0.31%     
==========================================
  Files           8        8              
  Lines        1634     1664      +30     
==========================================
+ Hits         1512     1545      +33     
+ Misses        122      119       -3
Impacted Files Coverage Δ
pySOT/__init__.py 100% <100%> (ø) ⬆️
pySOT/strategy.py 89.7% <90.12%> (-0.04%) ⬇️
pySOT/surrogate.py 93.43% <0%> (+2%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00eabbd...b8e7473. Read the comment docs.

@dme65 dme65 merged commit d1d0509 into master Jul 5, 2019
@dme65 dme65 deleted the WIP_restarts branch July 5, 2019 03:05
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