-
Notifications
You must be signed in to change notification settings - Fork 246
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
SkewMultivariateNormal
and SkewMultivariateStudentT
#1452
Comments
@colehaus yes, i'm sure a PR would be welcome. Would it help to use tfp.math.betainc?
|
Unless I'm misunderstanding you: There's a comment in the source describing the problem there which is that |
FYI since the last release, tfp.math.betainc has grad w.r.t. all parameters. I would suggest to have 3 PRs for:
|
Ah, that's good news! And that sounds like a reasonable plan. I probably won't be able to think about doing it for a few weeks though. |
I have implemented versions of both of these:
(I also have some coding testing them.)
SkewMultivariateStudentT
is notably slower thanMultivariateStudentT
in some circumstances. Are there any obvious performance improvements available?The text was updated successfully, but these errors were encountered: