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

Fix module import and naming in MNIST CNN example #12821

Closed
wants to merge 1 commit into from

Conversation

srkiNZ84
Copy link

Summary

With the new version of Tensorflow (2.0.0alpha), it seems that the Keras module names have changed. Updating the MNIST CNN (and possibly others might need to be updated as well) example to reflect this.

Was found after reading through: #12379

Seems quite a straightforward thing to fix, not sure why nobody has committed patches.

Tested with Python 3.6.7, Tensorflow 2.0.0a0 and CUDA 10.1

Related Issues

#12379

PR Overview

  • [n] This PR requires new unit tests [y/n] (make sure tests are included)
  • [n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
  • [n] This PR is backwards compatible [y/n]
  • [n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Was found after reading through: keras-team#12379

Seems quite a straightforward thing to fix, not sure why nobody has committed patches.

Tested with Python 3.6.7, Tensorflow 2.0.0a0 and CUDA 10.1
@fchollet
Copy link
Collaborator

For the time being, the examples in this repo should import keras rather than use from tensorflow import keras.

@fchollet fchollet closed this May 15, 2019
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