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

conv_encoder function Differs from Beta-VAE Paper? #37

Closed
nmichlo opened this issue Nov 23, 2021 · 1 comment
Closed

conv_encoder function Differs from Beta-VAE Paper? #37

nmichlo opened this issue Nov 23, 2021 · 1 comment

Comments

@nmichlo
Copy link

nmichlo commented Nov 23, 2021

Based on row 3 of Table 1 on page 13 of "beta-VAE: Learning Basic Visual
Concepts with a Constrained Variational Framework"
(https://openreview.net/forum?id=Sy2fzU9gl)

The Beta-VAE paper states that all the convolutional kernels are of size 4x4. I am not sure if it is intended that the 3rd and 4th convolutional layers in the conv_encoder function reduce these kernel sizes to 2x2?

e3 = tf.layers.conv2d(
inputs=e2,
filters=64,
kernel_size=2,
strides=2,
activation=tf.nn.relu,
padding="same",
name="e3",
)
e4 = tf.layers.conv2d(
inputs=e3,
filters=64,
kernel_size=2,
strides=2,
activation=tf.nn.relu,
padding="same",
name="e4",
)

@nmichlo
Copy link
Author

nmichlo commented Nov 23, 2021

An open issue already exists. Sorry. #23

@nmichlo nmichlo closed this as completed Nov 23, 2021
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