SacessOptimizer: More frequent cooperation

[dweindl]

Check during every ESS iteration if cooperation/adaptation critera are met.

[codecov-commenter]

Codecov Report

Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (4a953cd) 82.88% compared to head (261cf82) 53.18%.

❗ Current head 261cf82 differs from pull request most recent head 1c5cd3e. Consider uploading reports for the commit 1c5cd3e to get more accurate results

Files	Patch %	Lines
pypesto/optimize/ess/refset.py	5.00%	19 Missing ⚠️
pypesto/optimize/ess/sacess.py	5.88%	16 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff              @@
##           develop    #1194       +/-   ##
============================================
- Coverage    82.88%   53.18%   -29.71%     
============================================
  Files          148      148               
  Lines        11724    11750       +26     
============================================
- Hits          9718     6249     -3469     
- Misses        2006     5501     +3495     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[PaulJonasJost]

I do not exactly see how the cooperation became more frequent, but otherwise looks fine. As far as I recall, the keep_going part shortened due to a previous PR?

[dweindl]

I do not exactly see how the cooperation became more frequent, but otherwise looks fine. As far as I recall, the keep_going part shortened due to a previous PR?

Previously, a full scatter search was run (incorrectly) instead of only a single iteration. The previous PR was some refactoring to enable performing only one single iteration.