Skip to content

Conversation

@danielturek
Copy link
Member

Not to be merged.

Just want to see what testing does at this point.

It would be a small miracle if this ever gets merged, due to implicit assumptions made in the conjugacy system regarding size / dimensionality, that have come to light.

@danielturek
Copy link
Member Author

Noting in advance that some upcoming pushes will cause testing to fail. WIP.

@danielturek
Copy link
Member Author

This PR makes conjugacy processing more general. Dependent nodes of target conjugate nodes no longer must have the same sizes as the target node. Rather, dependent nodes (and, their parameters) can have different sizes (and dimensions) from the target node.

In particular, this suports the conjugacy between a gamma prior for tau, when tau is used as the precision parameter in dcar_normal distributions. Support for this relationship is also added, as well as a test of the asymptotic correct distribution.

@paciorek @perrydv I hope to merge this. But it's a minor under-hauling of conjugacy processing. I know it's minor in the scheme of things. I welcome any feedback. I think it's sound.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants