-
Notifications
You must be signed in to change notification settings - Fork 608
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 tutorials references to ops docs - general section #3869
Add tutorials references to ops docs - general section #3869
Conversation
Signed-off-by: Albert Wolant <awolant@nvidia.com>
!build |
"tensor_join.ipynb", | ||
op_reference('fn.cat', "Tutorial describing tensor joining")), | ||
doc_entry( | ||
"reinterpret.ipynb", |
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 think you should add squeeze
, expand_dims
and reinterpret
as this tutorial describes them as well.
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.
Done
"reductions.ipynb", | ||
op_reference('fn.reductions', "Tutorial describing how to use reductions")), | ||
doc_entry( | ||
"tensor_join.ipynb", |
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 think you need to list stack
here as well.
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.
Done
CI MESSAGE: [4747915]: BUILD STARTED |
Signed-off-by: Albert Wolant <awolant@nvidia.com>
CI MESSAGE: [4747915]: BUILD PASSED |
op_reference('fn.transforms', "Tutorial describing tensor geometric transformations"), | ||
op_reference('fn.warp_affine', "Tutorial describing tensor geometric transformations"), | ||
op_reference('fn.coord_transform', "Tutorial describing tensor geometric transformations")]), |
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.
This is just point of view. Maybe we can a bit farther and write which particular aspect this tutorial is touching, like:
op_reference('fn.transforms', "Tutorial describing tensor geometric transformations"), | |
op_reference('fn.warp_affine', "Tutorial describing tensor geometric transformations"), | |
op_reference('fn.coord_transform', "Tutorial describing tensor geometric transformations")]), | |
op_reference('fn.transforms', "Tutorial describing tensor geometric transformations to transform points and images"), | |
op_reference('fn.warp_affine', "Tutorial showing how to use afine transform"), | |
op_reference('fn.coord_transform', "Tutorial describing how to transform points accompanying images")]), |
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.
Done
Signed-off-by: Albert Wolant <awolant@nvidia.com>
!build |
CI MESSAGE: [4750729]: BUILD STARTED |
CI MESSAGE: [4750729]: BUILD PASSED |
* Add tutorials references to ops docs Signed-off-by: Albert Wolant <awolant@nvidia.com>
* Add tutorials references to ops docs Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant awolant@nvidia.com
Category:
Refactoring: Add tutorials references to ops docs - general section
Description:
Add tutorials references to ops docs - general section
Additional information:
Affected modules and functionalities:
Docs
Key points relevant for the review:
Are proper ops referenced
Checklist
Tests
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: DALI-2718