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

Change of Keras API #12

Open
Vopaaz opened this issue Aug 31, 2019 · 2 comments
Open

Change of Keras API #12

Vopaaz opened this issue Aug 31, 2019 · 2 comments

Comments

@Vopaaz
Copy link

Vopaaz commented Aug 31, 2019

When I run pix2pix-keras.ipynb, in the fifth block

if K._BACKEND == 'theano':

raises

AttributeError: module 'keras.backend' has no attribute '_BACKEND'

In the current Keras version (2.2.5), the proper API has changed to K.backend().
Would you please fix that, or should I draft a PR?

Besides, consider that the repo is created 2-3 years ago, there may be a lot of such API change and will be troublesome to fix them all. Would it be helpful to add a requirements.txt stating all the proper version of used packages, to avoiding such problems?

Thanks!

@tjwei
Copy link
Owner

tjwei commented Sep 2, 2019

Thanks, I will modernize the code soon, and move old code to a legacy folder and hopefully make a requirements.txt for them.

@Vopaaz
Copy link
Author

Vopaaz commented Sep 2, 2019

Much thanks!

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