tensorflowbook icon indicating copy to clipboard operation
tensorflowbook copied to clipboard

migrate tensorflow 1.0 or higher

Open BGPark opened this issue 8 years ago • 7 comments

Migrate some examples to the new API (1.0 or later). I have confirmed that my modifications work with TF versions 1.0 and 1.1. However, the remaining uncommitted examples (araxiv and CNNs examples) still have problems with the new API.

It is not perfect, but I hope it helps others.

BGPark avatar May 05 '17 08:05 BGPark

@backstopmedia / @samjabrahams - please update this - or specify hard version with pip install requirements to older tensorflow.

johndpope avatar Jun 22 '17 15:06 johndpope

Thanks, BGPark. I was able to adjust much of the code to suit the new API but your code is making my life much easier. Hopefully, the authors will update their code.

michael-GitHub-0 avatar Jul 13 '17 19:07 michael-GitHub-0

@michael-GitHub-0 Did all examples work for you with this patch?

danijar avatar Jul 14 '17 20:07 danijar

@danijar I am sorry, I haven't tried all of the code by BGPark. I only found this while working on chapter 5, and then I only pop over if I get stuck. I am new to GitHub.

michael-GitHub-0 avatar Jul 14 '17 22:07 michael-GitHub-0

It would be really nice if all examples are fixed under tf 1.0 or higher.

penchy-zju avatar Jul 26 '17 07:07 penchy-zju

The book started out very well, but something changed beginning with chapter 5. The writing was less explanatory (more typos, too) and more difficult to work with. I have made a few GitHub posts and sometimes the authors respond, and sometimes they don't. I have moved on to another book, myself.

michael-GitHub-0 avatar Jul 26 '17 19:07 michael-GitHub-0

Thank you For your work !

CosmosNing avatar Aug 11 '17 09:08 CosmosNing