-
Notifications
You must be signed in to change notification settings - Fork 348
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
add docs for all tutorial #12
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Links on the homepage jump to the github. It may be more reasonable to jump to the corresponding page of the sphinx doc
- The API document content tree on the homepage is duplicated with the sidebar. Maybe it would be better to hide the API document content tree on the homepage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please see inline comments
Done. Please @pan-x-c have a look and check this commit and this commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Checkpoint bug fixed and change magic pot to realistic concept
name: added the tutorials
about: improving the internal tutorial drafts and update them into the github
Description
[Please describe the background, purpose, changes made, and how to test this PR]
The modifications mainly include term unification, typo fixes, hyper-links update, and re-ordering and simplifying some sections for a better flow. Specific diff can be found in internal yuque.
Checklist
Please check the following items before code is ready to be reviewed.