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

Replace deprecated FastGFile with GFile #24456

Merged
merged 2 commits into from
Jan 8, 2019

Conversation

yongtang
Copy link
Member

FastGFile has been deprecated and replaced with GFile, though
the example in speech_commands still uses FastGFile. This fix
fix the issue to remove the deprecated warning:

WARNING:tensorflow:From <stdin>:1: __init__ (from tensorflow.python.platform.gfile) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.gfile.GFile.

Signed-off-by: Yong Tang yong.tang.github@outlook.com

FastGFile has been deprecated and replaced with GFile, though
the example in speech_commands still uses FastGFile. This fix
fix the issue to remove the deprecated warning:
```
WARNING:tensorflow:From <stdin>:1: __init__ (from tensorflow.python.platform.gfile) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.gfile.GFile.
```

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@ymodak ymodak self-assigned this Dec 20, 2018
@ymodak ymodak requested a review from yifeif December 20, 2018 00:18
@ymodak ymodak added the awaiting review Pull request awaiting review label Dec 20, 2018
@dksb dksb added this to Assigned Reviewer / Pending Review in PR Queue Dec 20, 2018
@dksb dksb added the size:XS CL Change Size: Extra Small label Dec 22, 2018
@tensorflowbutler
Copy link
Member

Nagging Reviewer @yifeif: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jan 8, 2019
@yifeif yifeif added ready to pull PR ready for merge process and removed awaiting review Pull request awaiting review labels Jan 8, 2019
@tensorflow-copybara tensorflow-copybara merged commit 4696da4 into tensorflow:master Jan 8, 2019
PR Queue automation moved this from Approved by Reviewer to Merged Jan 8, 2019
tensorflow-copybara pushed a commit that referenced this pull request Jan 8, 2019
@yongtang yongtang deleted the 12192018-FastGFile branch January 8, 2019 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes ready to pull PR ready for merge process size:XS CL Change Size: Extra Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

7 participants