[Add] Resize Operator
Added resize operator.
Merge request reports
Activity
added 2 commits
added 14 commits
-
b24ec71e...17948093 - 10 commits from branch
eclipse/aidge:dev
- 38b0dc3d - Added asserts for testing the same input dimension.
- 37d6becf - comments added and cleaning
- 6a8c362d - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cpu into operator_resize
- 438823d5 - included added
Toggle commit list-
b24ec71e...17948093 - 10 commits from branch
added 6 commits
-
cd37b514...3988b2f4 - 5 commits from branch
eclipse/aidge:dev
- 856ea6cf - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cpu into operator_resize
-
cd37b514...3988b2f4 - 5 commits from branch
added 8 commits
-
856ea6cf...583195c7 - 6 commits from branch
eclipse/aidge:dev
- 50f68929 - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cpu into HEAD
- fcb63863 - update resize new implementation clean version
-
856ea6cf...583195c7 - 6 commits from branch
62 62 } 63 63 }}); 64 64 65 std::shared_ptr<Tensor> roi = std::make_shared<Tensor>(); 66 std::shared_ptr<Tensor> scales = std::make_shared<Tensor>(); 65 67 std::shared_ptr<Tensor> sizes = std::make_shared<Tensor>(Array1D<float, 4>{ 66 68 {1, 1, 4 , 4}}); 67 69 68 std::size_t sizes_length = 4; 69 70 70 std::shared_ptr<Node> myResize = Resize("myresize_name_noneeded"); 71 71 auto op = std::static_pointer_cast<OperatorTensor>(myResize -> getOperator()); 72 72 op -> associateInput(0, myInput); 73 // op -> associateInput(1, roi); 74 // op -> associateInput(2, scales); 73 op -> associateInput(1, roi); First of all according to the ONNX documentation: https://github.com/onnx/onnx/blob/main/docs/Operators.md#inputs-1---4
roi
is optional so you don't always need to associate it.Second of all, why would you associate an unitialized tensor as an input ? You would be better off just not associating it
changed this line in version 9 of the diff
63 63 }}); 64 64 65 std::shared_ptr<Tensor> roi = std::make_shared<Tensor>(); 66 std::shared_ptr<Tensor> scales = std::make_shared<Tensor>(); 65 67 std::shared_ptr<Tensor> sizes = std::make_shared<Tensor>(Array1D<float, 4>{ 66 68 {1, 1, 4 , 4}}); 67 69 68 std::size_t sizes_length = 4; 69 70 70 std::shared_ptr<Node> myResize = Resize("myresize_name_noneeded"); 71 71 auto op = std::static_pointer_cast<OperatorTensor>(myResize -> getOperator()); 72 72 op -> associateInput(0, myInput); 73 // op -> associateInput(1, roi); 74 // op -> associateInput(2, scales); 73 op -> associateInput(1, roi); 74 op -> associateInput(2, scales); changed this line in version 9 of the diff
120 120 } 121 121 }}); 122 122 123 std::shared_ptr<Tensor> roi = std::make_shared<Tensor>(); 124 123 125 std::shared_ptr<Tensor> scales = std::make_shared<Tensor>(Array1D<float, 4>{ 124 126 {1, 1, 2, 2}}); 125 127 126 std::size_t sizes_length = 4; 127 128 128 std::shared_ptr<Node> myResize = Resize("myresize_name_noneeded"); 129 129 auto op = std::static_pointer_cast<OperatorTensor>(myResize -> getOperator()); 130 130 op -> associateInput(0, myInput); 131 // op -> associateInput(1, roi); 131 132 op -> associateInput(1, roi); changed this line in version 9 of the diff
@mszczep have looked into @olivierbichler comments?
Yes, I have discussed this issue with @olivierbichler . It appears that the problem can be solved in the onnx import.
assigned to @mszczep
requested review from @olivierbichler
added Feature 🚀 TopicOperator Unit-test labels
added 94 commits
-
d1b7a339...69c994fe - 85 commits from branch
eclipse/aidge:dev
- 7a613ff4 - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cpu into operator_resize
- ebac55d7 - Merge remote-tracking branch 'mszczep/operator_resize' into dev
- 9afaa93e - Merge branch 'dev' into operator_resize
- 9e989dca - header added
- f634c91f - tests added and bug fixed
- 3971ebd6 - conditions renames
- 55b79617 - old file remove
- 68beb0cb - float scales added
- 1fffaa50 - cleaning prints
Toggle commit list-
d1b7a339...69c994fe - 85 commits from branch