-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Fixes for broken links in documentation #2828
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.
It seems copyright update to 2025 are missing.
51e4f1c
to
c92d6a2
Compare
@ranukund, please review |
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.
I reviewed the changes in graph examples.
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.
Reviewed the changes. Thanks!
Some examples define 'tag' and 'df' shortcuts to improve code readability. This detail is misleading when looking at annotated examples code that only shows code snippets.
c1ccb48
to
96467d2
Compare
Fixes broken links:
tag::nchw
,tag::iohw
, andtag::any
.Also removed
dt
andtag
namespace shortcuts as these create confusion in annotated examples like this one.