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

Keras #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Keras #4

wants to merge 2 commits into from

Conversation

carlthome
Copy link

This pull request is not ready for merge yet, as I'm still having some issues. I'll notify you when there are updated results in the ipynb.

Hi @craffel, I'd like to add Keras to the benchmark if that's alright? I think it will be very interesting to see the difference between Lasagne and Keras considering how both use Theano. They should perform identically.

@carlthome carlthome mentioned this pull request Mar 22, 2016
7 tasks
@craffel
Copy link
Owner

craffel commented Mar 23, 2016

Cool, thanks! I assume you do not have results yet because of the netcdf issue? Some higher-level comments:

  • It is useful to check that the # of parameters is the same as for the other libraries. Not sure how Keras does this.
  • If you don't mind, please follow PEP-8 (put imports at the top of the file, shorten lines to 80 characters, etc)

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.

2 participants