-
Notifications
You must be signed in to change notification settings - Fork 615
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
TensorList shape #3591
TensorList shape #3591
Conversation
Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
4ba743f
to
0f833e9
Compare
!build |
CI MESSAGE: [3635530]: BUILD STARTED |
CI MESSAGE: [3635530]: BUILD PASSED |
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.
Looks ok, please add a dedicated test for TensorList.
Also as a bonus question: can we impact the type of the return value shown in the docstring anyhow?
I wonder if we can make it -> list of tuples
or list[tuple]
(or however it is according to Python type annotations). (Just curious if it's possible, it would be nice to have but it's not a blocker).
@@ -693,6 +702,10 @@ void ExposeTensorList(py::module &m) { | |||
.def("layout", [](TensorList<CPUBackend> &t) { | |||
return t.GetLayout().str(); | |||
}) | |||
.def("shape", &py_shape_list<CPUBackend>, |
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.
Can you add a test for this in the test_backend_impl.py or similar? Produce a TensorList of known shape and check if it returns it?
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.
Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
!build |
CI MESSAGE: [3669100]: BUILD STARTED |
CI MESSAGE: [3669100]: BUILD FAILED |
Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
!build |
CI MESSAGE: [3672611]: BUILD STARTED |
Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
!build |
CI MESSAGE: [3672877]: BUILD STARTED |
CI MESSAGE: [3672877]: BUILD PASSED |
Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
!build |
CI MESSAGE: [3673161]: BUILD STARTED |
CI MESSAGE: [3673161]: BUILD FAILED |
CI MESSAGE: [3673161]: BUILD PASSED |
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Add shape to TensorListCPU/GPU and DataNodeDebug Signed-off-by: ksztenderski <ksztenderski@nvidia.com>
Description
What happened in this PR
Adds shape method to TensorListCPU/GPU and DataNodeDebug.
Additional information
TensorListCPU/GPU
DataNodeDebug
Checklist
Tests
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A