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

LKJL with TransformVariables #9

Closed
cscherrer opened this issue May 31, 2021 · 3 comments
Closed

LKJL with TransformVariables #9

cscherrer opened this issue May 31, 2021 · 3 comments

Comments

@cscherrer
Copy link

I'd like to adapt LKJL for MeasureTheory.jl. I have a proof of concept, but @sethaxen found an inconsistency (JuliaMath/MeasureTheory.jl#100), and I'm not sure of the right way to work around it.

LKJL in this package works with the lower Cholesky factor. TransformVariables has CorrCholeskyFactor that seems like it would work, but it works with the upper Cholesky factor.

Do you have a way of using TransformVariables with LKJL?

@sethaxen
Copy link

The lower and upper cholesky factors are just transposes of each other, so the CorrCholeskyFactor transformation can be used if followed by a transpose. Since transpose has a logabsdetjac of 0, there's no need to adjust the scoring function. Unless I've missed something.

@cscherrer
Copy link
Author

Good point @sethaxen , it should be very easy to do this and create a new transformation. @tpapp , I'll leave this open in case you have anything to add. If not, feel free to close the issue.

@tpapp
Copy link
Owner

tpapp commented Jun 1, 2021

@sethaxen is correct. I will just close this, but feel free to ping me here if anything comes up and I will reopen.

@tpapp tpapp closed this as completed Jun 1, 2021
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

No branches or pull requests

3 participants