Skip to content

Fixed CumSum dnn layer#24353

Merged
asmorkalov merged 4 commits intoopencv:4.xfrom
alexlyulkov:al/fixed-cumsum-layer
Oct 3, 2023
Merged

Fixed CumSum dnn layer#24353
asmorkalov merged 4 commits intoopencv:4.xfrom
alexlyulkov:al/fixed-cumsum-layer

Conversation

@alexlyulkov
Copy link
Copy Markdown
Contributor

@alexlyulkov alexlyulkov commented Oct 3, 2023

Fixes #20110

The algorithm had several errors, so I rewrote it.
Also the layer didn't work with non constant axis tensor. Fixed it.
Enabled CumSum layer tests from ONNX conformance.

@alexlyulkov
Copy link
Copy Markdown
Contributor Author

Fixes #20110

@alexlyulkov alexlyulkov changed the title Fixed CumSum dnn layer (algorithm and ONNX parser) WIP Fixed CumSum dnn layer (algorithm and ONNX parser) Oct 3, 2023
@alexlyulkov alexlyulkov changed the title WIP Fixed CumSum dnn layer (algorithm and ONNX parser) Fixed CumSum dnn layer Oct 3, 2023
@asmorkalov asmorkalov requested a review from dkurt October 3, 2023 05:52
@asmorkalov asmorkalov added this to the 4.9.0 milestone Oct 3, 2023
@asmorkalov asmorkalov assigned dkurt and unassigned asmorkalov Oct 3, 2023
Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Tested manually with Ubuntu 18.04.

@asmorkalov asmorkalov merged commit 9bd14d5 into opencv:4.x Oct 3, 2023
hanliutong pushed a commit to hanliutong/opencv that referenced this pull request Oct 7, 2023
Fixed CumSum dnn layer opencv#24353

Fixes opencv#20110

The algorithm had several errors, so I rewrote it.
Also the layer didn't work with non constant axis tensor. Fixed it.
Enabled CumSum layer tests from ONNX conformance.
@asmorkalov asmorkalov mentioned this pull request Oct 17, 2023
thewoz pushed a commit to thewoz/opencv that referenced this pull request Jan 4, 2024
Fixed CumSum dnn layer opencv#24353

Fixes opencv#20110

The algorithm had several errors, so I rewrote it.
Also the layer didn't work with non constant axis tensor. Fixed it.
Enabled CumSum layer tests from ONNX conformance.
thewoz pushed a commit to thewoz/opencv that referenced this pull request May 29, 2024
Fixed CumSum dnn layer opencv#24353

Fixes opencv#20110

The algorithm had several errors, so I rewrote it.
Also the layer didn't work with non constant axis tensor. Fixed it.
Enabled CumSum layer tests from ONNX conformance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't create layer "2840" of type "CumSum" in function 'getLayerInstance'

3 participants