[Good First Issue]: Extend ONNX FE with `Col2Im` operator

[p-wysocki]

Context

A framework frontend (ONNX Frontend in this example) is responsible for translating operations from other standards into OpenVINO. This task entails creating a translation layer for operator Col2Im - the behavior of how ONNX's operation is interpreted in OpenVINO needs to be defined.

Since OpenVINO's implementation of Col2Im was partially based on ONNX's, this should be a direct translation with no issues.

What needs to be done?

1. Extend ONNX FE with Col2Im based on linked reference PRs
2. Remove xfailed tests regarding Col2Im:
   

   openvino/src/frontends/onnx/tests/tests_python/test_backend.py

   Lines 415 to 419 in 4c248f6

   	"OnnxBackendNodeModelTest.test_col2im_5d_cpu",
   	"OnnxBackendNodeModelTest.test_col2im_cpu",
   	"OnnxBackendNodeModelTest.test_col2im_dilations_cpu",
   	"OnnxBackendNodeModelTest.test_col2im_pads_cpu",
   	"OnnxBackendNodeModelTest.test_col2im_strides_cpu",

3. Assert the tests are now passing

Example Pull Requests

#15076
#22000
#14936
#12510
More can be found when searching "Extend ONNX" in the PRs

Resources

• Contribution guide - start here!
• Intel DevHub Discord channel - engage in discussions, ask questions and talk to OpenVINO developers
• How to link your Pull Request to an issue
• OpenVINO Col2Im docs
• ONNX Col2Im docs

Contact points

@p-wysocki
@gkrivor
@mitruska

AI notice - Important!

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact.

Ticket

CVS-99952

[1shChheda]

Hello,

I'm interested in making my first contribution to OpenVINO by working on issue #30144, which involves extending the ONNX frontend to support the Col2Im operator.

From what I gather, the task involves:

• Creating a new file, like col2im.cpp, in the src/frontends/onnx/frontend/src/op directory.
• Implementing a translation function in this file to map the ONNX Col2Im inputs and attributes to OpenVINO’s internal Col2Im operation.
• Registering the operator for the appropriate opset version, likely in ops_bridge.cpp.
• Updating the tests by removing the xfailed Col2Im tests in test_backend.py and ensuring they pass.

I’ve reviewed the contribution guide and am familiar with the process of submitting a pull request.

I have a couple of quick questions:

• Is there a specific way you’d like me to handle attributes like padding in the translation?
• Are there any particular test cases I should prioritize beyond what’s already in the issue?

I'm eager to get started and would appreciate any guidance or feedback on my proposed approach. Please let me know if there’s anything else I should keep in mind. Thanks!

[sanleo-wq]

.take

[github-actions]

Thank you for looking into this issue! Please let us know if you have any questions or require any help.