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

filldist, up1 not defined #255

Closed
mikkoku opened this issue Apr 26, 2023 · 6 comments · Fixed by #259
Closed

filldist, up1 not defined #255

mikkoku opened this issue Apr 26, 2023 · 6 comments · Fixed by #259

Comments

@mikkoku
Copy link

mikkoku commented Apr 26, 2023

I hope this is the right package for this issue.
Running

using Turing
@model function test()
    beta ~ filldist(Normal(), 3, 2)
end
sample(test(), NUTS(), 10)

results in

ERROR: UndefVarError: up1 not defined
Stacktrace:
  [1] bijector(d::DistributionsAD.MatrixOfUnivariate{Continuous, Normal{Float64}, FillArrays.Fill{Normal{Float64}, 2, Tuple{Base.OneTo{Int64}, Base.OneTo{Int64}}}})
    @ Bijectors C:\Users\03052156\.julia\packages\Bijectors\vKGbw\src\compat\distributionsad.jl:17
@torfjelde
Copy link
Member

Can you do

]st --manifest

and show us the versions of Turing.jl and Bijectors.jl?

@mikkoku
Copy link
Author

mikkoku commented May 5, 2023

  [76274a88] Bijectors v0.12.3
  [fce5fe82] Turing v0.25.0

@thevolatilebit
Copy link

Experiencing the same issue with

  [76274a88] Bijectors v0.12.4
  [fce5fe82] Turing v0.25.1

torfjelde added a commit that referenced this issue Jun 3, 2023
* fix for #255 and introduction of columnwise

* added some tests

* version  bump

* forgot to add the inverse
@torfjelde
Copy link
Member

This should now be fixed once you upgrade to [email protected]:)

@thevolatilebit
Copy link

@torfjelde I can confirm that it works! thank you very much!

@patrickm663
Copy link

@torfjelde Thank you, I had been facing a similar issue. This solves it.

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 a pull request may close this issue.

4 participants