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

MNIST NNX Tutorial broken links #4394

Open
stergiosba opened this issue Nov 22, 2024 · 5 comments
Open

MNIST NNX Tutorial broken links #4394

stergiosba opened this issue Nov 22, 2024 · 5 comments

Comments

@stergiosba
Copy link

Links that lead to the Github code and a google colab notebook at the top of the MNIST NNX tutorial page in the docs are both broken.

@cgarciae
Copy link
Collaborator

Hey @stergiosba, the link you posted is an old version of the docs. This is the current version: https://flax.readthedocs.io/en/latest/mnist_tutorial.html

@stergiosba
Copy link
Author

@cgarciae Thanks for pointing that out. I see that Google takes me to version v8.3 of the docs and not the latest by default. I deleted my Firefox cache, but the issue remains. Is this a different problem on my end?

@cgarciae
Copy link
Collaborator

What is your search query? If its NNX try searching for just "Flax" instead in the future. We might have to try to do something about NNX leading to an older version and my personal github 😅

@stergiosba
Copy link
Author

I just searched for flax nnx on google. On my workstation, searching for Flax returns a plant :)

@8bitmp3
Copy link
Collaborator

8bitmp3 commented Nov 27, 2024

Since Flax NNX is the "default" Flax API, maybe we can amend the title in
https://flax.readthedocs.io/en/latest/mnist_tutorial.html to emphasize "Flax NNX" to improve search.
LMKWYT @cgarciae @IvyZX

The source tutorial docs are in https://github.com/google/flax/tree/main/docs_nnx

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 a pull request may close this issue.

3 participants