-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
VI using deprecated Aesara MRG sampler #4523
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
Comments
@brandonwillard should I use the master branch or V4 for development? |
We've been putting in PRs to the |
So yes, I'll open a PR to v4 branch:tada: |
Oh, wait, the change requested here could go toward I would prioritize |
@brandonwillard, running
|
I have aesara v2.0.2 there |
using aesara master results in the same |
I have errors like this
|
Yes, you should expect all sorts of errors. |
Does any of this change with the Numba backend? |
It shouldn't; in general, if we use Aesara, its backends should handle everything without requiring backend-specific logic at this level. |
It is not working with pymc 4: import pymc
with pm.Model():
x = pm.Normal("x")
pm.fit() WARNING (aesara.tensor.blas): Using NumPy C-API based implementation for BLAS functions. |
@bukson please open an issue on our discourse at: https://discourse.pymc.io/ |
No this is a legit issue, we should port VI to use random variable: aesara-devs/aesara#322 (comment). CC @ferrine |
The current usage of the mrng sampler in ADVI does not support JAX (pymc-devs/pytensor#322). It should be fairly easy to instead make it use the new
RandomVariable
Op pymc-devs/pytensor#296 to get JAX support for ADVI.CC @ferrine
The text was updated successfully, but these errors were encountered: