Skip to content

JA: add ja/guide/eager.ipynb#999

Merged
TensorFlow-Docs-Copybara merged 6 commits into
tensorflow:masterfrom
chie8842:eager_r2
Sep 17, 2019
Merged

JA: add ja/guide/eager.ipynb#999
TensorFlow-Docs-Copybara merged 6 commits into
tensorflow:masterfrom
chie8842:eager_r2

Conversation

@chie8842
Copy link
Copy Markdown
Contributor

#752 is closed, so I fixed the PR comment and re-open.

CC. @sfujiwara

@tfdocsbot tfdocsbot added ja translation Community-provided translation for certain languages labels Sep 14, 2019
@tfdocsbot
Copy link
Copy Markdown
Collaborator

Preview and run these notebook edits with Google Colab:

Notebook diffs available on ReviewNB.com.

@tfdocsbot
Copy link
Copy Markdown
Collaborator

Reviewers added, please take a look.
@ohtaman, @sfujiwara, @masa-ita, @AseiSugiyama, @yukinagae

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

@googlebot googlebot added the cla: yes CLA has been signed label Sep 14, 2019
Copy link
Copy Markdown
Contributor

@sfujiwara sfujiwara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinks and LGTM!

@tfdocsbot tfdocsbot added kokoro:force-run lgtm Community-added approval labels Sep 15, 2019
Copy link
Copy Markdown
Member

@lamberta lamberta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

Fixed the merge conflict and also updated the notebook that was breaking. Converted tf.random_normal to tf.random.normal for TF2. Please make sure to test the notebook in Colab to make sure it works :)

@lamberta lamberta added the ready to pull Start merge process label Sep 17, 2019
TensorFlow-Docs-Copybara pushed a commit that referenced this pull request Sep 17, 2019
PiperOrigin-RevId: 269601907
@TensorFlow-Docs-Copybara TensorFlow-Docs-Copybara merged commit 7ff0e99 into tensorflow:master Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes CLA has been signed ja lgtm Community-added approval ready to pull Start merge process translation Community-provided translation for certain languages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants