You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is an inconsistency between the celerite2 Getting Started tutorial and the Exoplanet tutorials in the use of GaussianProcess and compute. Specifically, in the celerite2 tutorial compute is called explicitly but in the Exoplanet tutorials that include celerite2, compute is implicitly called through supplying the t keyword.
I don't think this is a particular problem, but it would be good if the main GaussianProcess docstring stated that if t is supplied then the compute method is automatically run.
In addition, the theano.GaussianProcess class doesn't have a docstring so this would benefit from having the same docstring as the main GaussianProcess class, also with the addition of the note about compute.
The text was updated successfully, but these errors were encountered:
There is an inconsistency between the celerite2 Getting Started tutorial and the Exoplanet tutorials in the use of
GaussianProcess
andcompute
. Specifically, in the celerite2 tutorialcompute
is called explicitly but in the Exoplanet tutorials that include celerite2,compute
is implicitly called through supplying thet
keyword.I don't think this is a particular problem, but it would be good if the main
GaussianProcess
docstring stated that ift
is supplied then the compute method is automatically run.In addition, the
theano.GaussianProcess
class doesn't have a docstring so this would benefit from having the same docstring as the mainGaussianProcess
class, also with the addition of the note about compute.The text was updated successfully, but these errors were encountered: