-
Notifications
You must be signed in to change notification settings - Fork 8
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
Switch to SimpleVarInfo
for better computational efficiency
#43
Comments
Does that change how we would write the model inside the |
It should just work once the |
@yebai has this been already merged into |
We are very close now - only one final PR standing. Hopefully, it will get reviewed and merged this week or next. |
Ok I subscribed to it and once it is merged and a new Turing version is tagged I will run tests. |
SimpleVarInfo
inDynamicPPL
provides significant speedup over the current implementation. We should consider switching to use it once TuringLang/DynamicPPL.jl#360 is merged.The text was updated successfully, but these errors were encountered: