semi-supervised-pytorch icon indicating copy to clipboard operation
semi-supervised-pytorch copied to clipboard

Norm. flows bug maybe

Open kiwi0fruit opened this issue 5 years ago • 1 comments

On this line the code:

qz = log_gaussian(z, mu, log_var) - sum(log_det_z)

The first one and log_det_z are of batch size dim but the sum(log_det_z) is a scalar.

Actually the sum was already taken here.

It looks like a bug but I'm not sure.

kiwi0fruit avatar Aug 09 '19 12:08 kiwi0fruit

I believe you might be right, but I don't know what implications it will have to fix it. I will check on it a later time.

Thanks

wohlert avatar Aug 12 '19 10:08 wohlert