-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Include conditional autoregression (CAR) as multivariate distribution #3689
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
That would be awesome! Please go ahead and I am happy to review it ;-) |
Implementing the
I'd like anyone's opinion on which road to go down, if any. I am also open to any suggestions for alternative ideas for implementing the random method. |
Hmm neither sound very satisfying, I suggest leave out the random method for now. |
I have been using the implementation of the CAR distribution provided in @junpenglao's notebook and have found that it works pretty well in practice both as a likelihood and as a latent variable prior. I have used it with >10k locations on a GPU with some good speedups for both ADVI and NUTS. It would be nice if it were included out-of-the-box with PyMC3; I have done some instructional classes on spatial models and it's clunky to have to refer students to the notebook for the implementation.
If there is interest, I would like to submit a PR that includes the CAR as a multivariate distribution in similar form to the implementation from the notebook.
The text was updated successfully, but these errors were encountered: