You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In chirho.robust, we're making extensive use of torch.func, especially the vectorization transform torch.func.vmap and forward and reverse-mode autodiff transforms torch.func.jvp/vjp.
Unfortunately, the autodiff transforms seem to be fundamentally incompatible with PyroModules that have constrained PyroParams, including most of the standard autoguides in pyro.infer.autoguide - the raw PyTorch Parameter tensors underlying the constrained PyroParams do not seem to receive or propagate gradients. It's not clear what exactly in Pyro is causing the problem or how to fix or work around it, although it at least doesn't seem specific to ChiRho.
This isn't immediately blocking other work, but without compatibility with PyroModules the practical utility of chirho.robust will be significantly diminished, so we should try to resolve this somehow before merging #398 into master.
The text was updated successfully, but these errors were encountered:
In
chirho.robust
, we're making extensive use oftorch.func
, especially the vectorization transformtorch.func.vmap
and forward and reverse-mode autodiff transformstorch.func.jvp
/vjp
.Unfortunately, the autodiff transforms seem to be fundamentally incompatible with
PyroModule
s that have constrainedPyroParam
s, including most of the standard autoguides inpyro.infer.autoguide
- the raw PyTorchParameter
tensors underlying the constrainedPyroParam
s do not seem to receive or propagate gradients. It's not clear what exactly in Pyro is causing the problem or how to fix or work around it, although it at least doesn't seem specific to ChiRho.This isn't immediately blocking other work, but without compatibility with
PyroModule
s the practical utility ofchirho.robust
will be significantly diminished, so we should try to resolve this somehow before merging #398 intomaster
.The text was updated successfully, but these errors were encountered: