Skip to content

I made some minor changes in order to make it a bit clearer. - #3

Open
jpcosec wants to merge 1 commit into
miyosuda:masterfrom
jpcosec:reordered
Open

I made some minor changes in order to make it a bit clearer.#3
jpcosec wants to merge 1 commit into
miyosuda:masterfrom
jpcosec:reordered

Conversation

@jpcosec

@jpcosec jpcosec commented May 24, 2018

Copy link
Copy Markdown

In model.py

  • Changed the object name to BVae instead of Vae
  • Changed how some layers and the loss were declared in order to make it graph readable in tensorboard.
  • Changed names to encoder, decoder, etc.
  • Separated Z network method from the encoder and the decoder.
  • Introduced histograms for the Z variable.

In main.py

  • Changed Vae call to Bvae.

Pd: Nice work, it works just fine.

…le in tensorboard. Also introduced a Z histogram.
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

Successfully merging this pull request may close these issues.

1 participant