Fix DMM example with --num-iafs > 0#734
Conversation
fritzo
left a comment
There was a problem hiding this comment.
I agree the automated example discovery logic is getting kludgey.
|
@fritzo - EDIT - I'll let it be for now, and have you and @martinjankowiak take care of that. |
Its not that bad yet. :) Just added a TODO to keep in mind that if we are messing around with that part too much, it might be worth revisiting. |
I am working on that in this branch https://github.com/uber/pyro/compare/torch-transformed-and-iaf |
Yeah, I will need this for HMC. Unfortunately, we can't mix |
We needed some shape fixes to TransformedDistribution to fix the DMM example with IAF. There is still the issue with NAN loss - #711 that seems to have been introduced much earlier (I checked until November 2017).
Fixes #721.