DynamicPPL targets: only link! the varinfo if using gradient-based samplers
Closed this issue · 3 comments
miguelbiron commented
Could save time and allocations by not going back and forth between spaces
miguelbiron commented
Drawback: will fail when any distribution in the constrained space lives in a submanifold of the ambient space; e.g., Dirichlet.
alexandrebouchard commented
Good point.. seems better not to do this then. Just to double-check, this was just a suggestion, was not actually implemented? Then I suggest we close that issue.
miguelbiron commented
Yep, not implemented. Closing now!