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

Fix multiple links to Orbax documentation #4364

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

Matt-Hurd
Copy link
Contributor

Orbax documentation was moved in google/orbax@e8b8baa. It was previously all in a single markdown file: checkpoint.md. The documentation has since been split into multiple ReadTheDocs pages.

What does this PR do?

Updates links to align with the new layout.

Checklist

  • This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
    checks if that's the case).
  • This change is discussed in a Github issue/
    discussion (please add a
    link).
  • The documentation and docstrings adhere to the
    documentation guidelines.
  • This change includes necessary high-coverage tests.
    (No quality testing = no merge!)

Orbax documentation was moved in google/orbax@e8b8baa. Updating links to align with their structure.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Collaborator

@8bitmp3 8bitmp3 left a comment

Choose a reason for hiding this comment

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

thank you @Matt-Hurd

@8bitmp3 8bitmp3 self-assigned this Nov 26, 2024
@8bitmp3 8bitmp3 requested a review from IvyZX November 26, 2024 19:10
@copybara-service copybara-service bot merged commit 21585ad into google:main Dec 10, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants