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

Update to new Tensorflow v1.0 API #5

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

Conversation

hofingermarkus
Copy link

Hi,
I've updated the files to work with the new Tensorflow API introduced in v1.0
I've tried to keep changes minimal
All files were able to execute without errors on my PC with GPU.

Regards Markus

 => tf.summary.scalar() does not accept placeholders as names, has to be a fixed python string => summary_label removed
Moving Itinerary.ipynb into Readme.md to have a better starting point on github.com
@samjabrahams
Copy link
Owner

samjabrahams commented Apr 29, 2017

Hi @hofingermarkus,

Thank you so much for putting in the time and effort to update the code in this repository. When I have some time (hopefully in the next 2-3 days), I'll run through the code to make sure it all works and merge this in.

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.

None yet

2 participants