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

Supervised concrete autoencoder #15

Open
annalauralerede opened this issue Aug 24, 2022 · 1 comment
Open

Supervised concrete autoencoder #15

annalauralerede opened this issue Aug 24, 2022 · 1 comment

Comments

@annalauralerede
Copy link

Hi,

I am looking through your code and I see that for the supersived implementation you use the MSE loss instead of the cross-entropy loss like you mentioned in the paper... There is no condition to use the cross-entropy loss when supervised and it always use the mse loss. Is it a mistake or is it done on purpose?

@mfbalin
Copy link
Owner

mfbalin commented Sep 17, 2023

One is free to use any kind of loss function for supervised training. It might be an oversight in the implementation.

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

2 participants