Conditional-PixelCNN-decoder icon indicating copy to clipboard operation
Conditional-PixelCNN-decoder copied to clipboard

Inconsistency with paper in connection from vstack to hstack

Open kkleidal opened this issue 7 years ago • 4 comments

https://github.com/anantzoid/Conditional-PixelCNN-decoder/blob/9a5c9a3df2c58100cf5e3600392e67db8ac7a59e/models.py#L30-L34

This has the connection from the vstack to the hstack occuring after the gate, but in Figure 2 of the paper, it occurs after the masked convolution but before splitting the channels and gating. Any particular reason for this change?

kkleidal avatar Nov 29 '17 18:11 kkleidal

Yeah I also found the issue. Is it the official code written by the original paper author?

falconjhc avatar Apr 04 '19 07:04 falconjhc

Sidenote: Sorry for my previous comment (now deleted). I didn't read the email closely enough and thought it was in response to a similarly named repo I made.

I don't think this is the official implementation. I did try to correct the issue in #7 which has since been merged.

kkleidal avatar Apr 04 '19 11:04 kkleidal

In the original paper, the link from vstack to hstack is before gate. This code is different.

yuffon avatar May 30 '19 09:05 yuffon

I'm new to this paper, and I found that the h_stack is different with the original paper. the code is"[filter_size if full_horizontal else 1, filter_size, conf.f_map]" and para "full_horizontal" is set as TRUE Cuz the size of mask is changging from 7x7 to 3x3, and perhaps the size of mask should be 1x7/1x3? I think the size of horizontal stack is changging from 1x3 to 1x7, but maybe I understand the wrong way, anyone can explain why the kernel changes from 7x7 to 3x3 and why it is 7x7 instead of 1x7? I'm so confused, plzplzplz help me out, I really appreciate your help! THX!!!!

Raaaaaaat avatar Dec 17 '19 11:12 Raaaaaaat