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

Autoencoder TODOs #12

Open
2 of 4 tasks
gdkrmr opened this issue Oct 11, 2017 · 2 comments
Open
2 of 4 tasks

Autoencoder TODOs #12

gdkrmr opened this issue Oct 11, 2017 · 2 comments

Comments

@gdkrmr
Copy link
Owner

gdkrmr commented Oct 11, 2017

TODOs for the Autoencoder:

  • write tests for the other activation function types
  • better stopping criteria
  • more than one training step, continue training a dimRedResult object
  • handle the installation of tensorflow and keras (this should be handled by the user -> IGNORE)
@topepo
Copy link
Contributor

topepo commented Oct 11, 2017

(sorry, I should have put my other comments in your repo)

Can you name the resulting variable differently? They aren't really related to PCA. Maybe use the prefix "AE" or similar?

@gdkrmr
Copy link
Owner Author

gdkrmr commented Oct 12, 2017

  • I did the renaming, autoencoders are called nl-PCA in literature related to Meteorology, but autoencoder is probably the better name.
  • I did a quick search and I am not so happy with using Keras, mainly, because it seems that it does not allow the use of shared weights, there are autoencoders where the decoder weights consist of the transposed encoder weights and a couple of other fancy things that seem difficult or impossible in Keras.
  • On the upside I think users could supply their own graph in Keras which would make something like convolutional autoencoders possible without me having to implement every special case and serialization seems super easy.

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