Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Encoder architecture does not match paper #23

Open
alexrakowski opened this issue Jan 27, 2020 · 0 comments
Open

Encoder architecture does not match paper #23

alexrakowski opened this issue Jan 27, 2020 · 0 comments

Comments

@alexrakowski
Copy link

Is the encoder architecture ( conv_encoder ) from here the one used in the experiments in the paper?
It seems to be a discrepancy in terms of the size of conv. kernels used in the 3rd and 4th layers.
The code uses 2x2 convolutions there, while, according to Table 2 from the Appendix of the paper 4x4 convolutions are used all the way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant