-
Notifications
You must be signed in to change notification settings - Fork 34
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
Skellam (special case) is broken with Zygote and SpecialFunctions >= 1 #156
Comments
Copy from TuringLang/DistributionsAD.jl#145 (comment):
|
So I guess this can be closed? |
No, Skellam is still broken with Zygote in one test. |
Do you know which one? I just had to remove a bunch of |
Are you sure? I fixed these in #182, I thought. |
Ah, it seems you updated the tests before merging the master branch. I updated this issue here when I fixed the tests in #182, so the remaining error is still not fixed. |
Aaaah, my bad; thanks! |
Currently, these distributions are broken due to FluxML/Zygote.jl#873. As discussed in #155, the corresponding tests are marked as broken. The issue will be fixed upstream.
See also: TuringLang/DistributionsAD.jl#145
The text was updated successfully, but these errors were encountered: