-
Notifications
You must be signed in to change notification settings - Fork 29
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
Allow non-Real values in UntypedVarInfo #107
Comments
I would like to see this a lot too -- I could imagine cases where someone had a distribution across bigrams or something and just wanted to work with the actual draws from that distribution. It seems to me the hangup here is not really so much in DynamicPPL but in MCMCChains and various other spots in the Turing-verse. |
Yep, it would be nice if we wouldn't make any assumptions about the support of the distributions we work with since we just need to be able to sample and evaluate the log pdf. |
Super exciting. I'm all in for this. |
We probably don't need this anymore due to the planned transition to |
It would be nice to allow arbitrary types in
UntypedVarInfo
instead of justReal
s. This should allow Turing to be used with distributions that sample letters, words, trees, images, etc. without having to represent them as an array of reals. The conversion toTypedVarInfo
should work fine whatever type is inUntypedVarInfo
so this may be as simple as changingReal
toAny
in a bunch of places.The text was updated successfully, but these errors were encountered: