Skip to content

Update autodiff.ipynb #2025

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

Closed
wants to merge 3 commits into from
Closed

Update autodiff.ipynb #2025

wants to merge 3 commits into from

Conversation

Saduf2019
Copy link
Contributor

Updated broken kink leading to 404 for the "basic guide-eager execution"

Updated broken kink leading to 404 for the "basic guide-eager execution"
8bitmp3
8bitmp3 previously approved these changes Feb 15, 2022
@8bitmp3 8bitmp3 self-assigned this Feb 15, 2022
@github-actions github-actions bot added the lgtm Community-added approval label Feb 15, 2022
@8bitmp3 8bitmp3 added ready to pull Start merge process lgtm Community-added approval and removed lgtm Community-added approval labels Feb 15, 2022
@@ -75,7 +75,7 @@
"[backpropagation](https://en.wikipedia.org/wiki/Backpropagation) for training\n",
"neural networks.\n",
"\n",
"In this guide, you will explore ways to compute gradients with TensorFlow, especially in [eager execution](eager.ipynb)."
"In this guide, you will explore ways to compute gradients with TensorFlow, especially in [eager execution](basics.ipynb)."
Copy link
Member

Choose a reason for hiding this comment

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

I think this is better if it's not a link at all. The target page doesn't mention eager execution & I can't find a decent alternative, so I think it's better to just leave the text, no link.

copybara-service bot pushed a commit that referenced this pull request Feb 23, 2022
PiperOrigin-RevId: 430364817
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Community-added approval ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants