secondmind-labs / trieste

A Bayesian optimization toolbox built on TensorFlow
Apache License 2.0
221 stars 42 forks source link

Make ensure_variable_model_data optional #752

Closed uri-granta closed 1 year ago

uri-granta commented 1 year ago

741 added a GaussianProcessRegression abstract method _ensure_variable_model_data. This is an unnecessary breaking change, as some subclasses may not need to implement this (and any that do should know about it by failing garden path tests). This PR makes it optional instead.