Tweaked TR initialization algorithm to not take the y_min value from the initial dataset, and instead leave it at inf. The previous version had the inconsistency that at initialisation the saved y_min value was not at the region center/location. This change means that the first update/step will always be successful.
Discussed and agreed this change with @vpicheny.
Fully backwards compatible: yes
PR checklist
[X] The quality checks are all passing
[ ] The bug case / new feature is covered by tests
[ ] Any new features are well-documented (in docstrings or notebooks)
Related issue(s)/PRs: None
Summary
Tweaked TR initialization algorithm to not take the
y_minvalue from the initial dataset, and instead leave it atinf. The previous version had the inconsistency that at initialisation the savedy_minvalue was not at the region center/location. This change means that the first update/step will always be successful.Discussed and agreed this change with @vpicheny.
Fully backwards compatible: yes
PR checklist