Skip to content

Update to new Tensorflow v1.0 API#5

Open
hofingermarkus wants to merge 6 commits intosamjabrahams:masterfrom
hofingermarkus:master
Open

Update to new Tensorflow v1.0 API#5
hofingermarkus wants to merge 6 commits intosamjabrahams:masterfrom
hofingermarkus:master

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.

2 participants