Always test at least one BO step even if the initial points happen to have found the minimum!
Increment max number of steps for a few failing tests (these are failing just due to rng changes and the early_stop_callback will prevent unnecessary steps anyway)
Remove unnecessary breaking API change: keep fit_initial_model and make fit_model a simpler bool.
Fully backwards compatible: yes
PR checklist
[x] The quality checks are all passing
[x] The bug case / new feature is covered by tests
[ ] Any new features are well-documented (in docstrings or notebooks)
Related issue(s)/PRs: https://github.com/secondmind-labs/trieste/pull/739
Summary
Fix failures introduced by TURBO PR:
fit_initial_model
and makefit_model
a simpler bool.Fully backwards compatible: yes
PR checklist