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

compatibility update, port data load code to tf.data, etc. #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wogong
Copy link
Contributor

@wogong wogong commented Aug 31, 2018

minor changes to make this repo be compatible with tf 1.10.

  1. port data load code to tf.data, https://github.com/tensorflow/tensorflow/blob/r1.4/tensorflow/contrib/data/README.md
  2. remove warning, "softmax_cross_entropy_with_logits deprecated", softmax_cross_entropy_with_logits deprecated tensorflow/minigo#37

@Prem95
Copy link

Prem95 commented Sep 2, 2018

For anyone not sure how to make the changes.

  1. Use data = tf.data.Dataset instead of tf.contrib.data. Same goes to Iterator
  2. Use tf.nn.softmax_cross_entropy_with_logits_v2 instead of tf.nn.softmax_cross_entropy_with_logits

@kratzert
Copy link
Owner

kratzert commented Sep 2, 2018

I'll look over this during the next week. Have some trouble with our newborn at the moment and I don't find much time to cross check the PR. But will come back to this, I promise

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.

3 participants