-
Notifications
You must be signed in to change notification settings - Fork 51
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
Poisson likelihood #231
Poisson likelihood #231
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! Two things that would be nice:
- A test to check that a non-conjugate GP is returned from taking the product of the prior and Poisson likelihood.
- A notebook demonstrating this. Simulated data would be fine and it could heavily draw on the MCMC routine used in the classification notebook.
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## v0.6 #231 +/- ##
==========================================
+ Coverage 87.04% 87.11% +0.07%
==========================================
Files 51 51
Lines 1961 1964 +3
Branches 183 184 +1
==========================================
+ Hits 1707 1711 +4
+ Misses 187 186 -1
Partials 67 67
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Adds the Poisson likelihood, along with an example notebook demonstrating a GP regression with count data.
Pull request type
Please check the type of change your PR introduces: