Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct log-likelihood docstrings and make them consistent #1512

Open
k-shep opened this issue Dec 12, 2023 · 0 comments
Open

Correct log-likelihood docstrings and make them consistent #1512

k-shep opened this issue Dec 12, 2023 · 0 comments

Comments

@k-shep
Copy link
Contributor

k-shep commented Dec 12, 2023

The Gaussian log-likelihood description (https://pints.readthedocs.io/en/stable/log_likelihoods.html#pints.GaussianLogLikelihood) in the MultiOutputProblem case is incorrect. There should be i dependence in the final sum.

The log-likelihood descriptions are also inconsistent between different log-likelihoods. For instance, the MultiplicativeGaussianLogLikelihood (https://pints.readthedocs.io/en/stable/log_likelihoods.html#pints.MultiplicativeGaussianLogLikelihood) has the i, j in different orders compared to the Gaussian log-likelihood documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant