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

zh-CN tutorials/estimators/linear #905

Open
wants to merge 4 commits into
base: master
from

Conversation

@ddreset
Copy link
Contributor

commented Aug 5, 2019

Please review.

@tfdocsbot

This comment has been minimized.

Copy link
Collaborator

commented Aug 5, 2019

Preview and run these notebook edits with Google Colab:

Notebook diffs available on ReviewNB.com.
@tfdocsbot

This comment has been minimized.

Copy link
Collaborator

commented Aug 5, 2019

Reviewers added, please take a look.
@kuri-leo, @JayYip, @yantaozhao, @loveunk, @Wind2esg, @tigerneil, @MofiiTech, @gaoljhy, @Mr-Linus, @flopsySong, @echosun1996

When your review is finished, approve the pull request or include "LGTM" in your comment.

@lamberta
Copy link
Member

left a comment

Thanks. Some files have moved around as we switch the site to TF2-first. Can you rebase and update this to match https://github.com/tensorflow/docs/blob/master/site/en/tutorials/estimator/linear.ipynb ?
(and update links, please)

@lamberta lamberta added the invalid label Sep 9, 2019

ddreset added 3 commits Aug 5, 2019

@ddreset ddreset force-pushed the ddreset:beta/tutorials/estimators/linear branch from 0a92952 to 11c5db3 Sep 10, 2019

@ddreset ddreset requested a review from lamberta Sep 10, 2019

@lamberta lamberta removed the invalid label Sep 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.