secondmind-labs / trieste

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

AskTellOptimizer needs an instances of TrainableProbabilisticModel #814

Closed pio-neil closed 8 months ago

pio-neil commented 8 months ago

The AskTellOptimizer needs instances of TrainableProbabilisticModel even if no training is being performed by the optimizer.

In other words, if only the ask method is called, and not the tell method, then having a trainable model seems unnecessary, but this is required by the type declaration.

In our case we train models outside of Trieste, so it would be nicer if we could specify just a ProbabilisticModel if we only intend to use the ask method, and not the tell method.