[ONNX] Aligned ReduceL1 behavior #20559

[RitikaxShakya]

Details:

-I've aligned the ReduceL1 operation with opset 11, 13, and 18. I've registered the function inside the ops_bridge.cpp, created test models, and added them inside onnx_import.in.cpp.

Tickets:

• [Good First Issue]: Align behavior of ONNX Frontend function ReduceL1-11, 13, 18 with original framework #20559

[RitikaxShakya]

@gkrivor @p-wysocki Please take a look at this PR. Thank You!

[gkrivor]

build_jenkins

[RitikaxShakya]

@gkrivor Hello! If there is anything i need to do from my end, please let me know. Thank you!

[github-actions]

This PR will be closed in a week because of 2 weeks of no activity.

[RitikaxShakya]

This PR will be closed in a week because of 2 weeks of no activity.

@gkrivor Hello! Please let me know If there is anything i need to do from my end. Thank you!

[mlukasze]

we are willing to merge it, could you please resolve conflicts?

[RitikaxShakya]

we are willing to merge it, could you please resolve conflicts?

Solved conflicts

[mlukasze]

build_jenkins

[github-actions]

This PR will be closed in a week because of 2 weeks of no activity.

[mlukasze]

build_jenkins

[mlukasze]

@gkrivor could you take a look, please?

[github-actions]

This PR will be closed in a week because of 2 weeks of no activity.

[mlukasze]

hey @RitikaxShakya
could you please resolve conflicts? we will review it very soon
thanks and sorry for the delay, it's on us :(

[RitikaxShakya]

hey @RitikaxShakya could you please resolve conflicts? we will review it very soon thanks and sorry for the delay, it's on us :(

Okay, i will try finishing it this week

[mlukasze]

thanks!

[gkrivor]

@RitikaxShakya could you rebase your change? There are several changes in the code which should be solved.

[gkrivor]

Hi @RitikaxShakya, tell me please if you have any issues with rebasing I'll help.

[gkrivor]

Hi @RitikaxShakya , any issues?

[RitikaxShakya]

Hi @RitikaxShakya , any issues?

Yes, i tried rebasing and solve conflicts but it kinda messed up by showing commits that i didn't make, i tried resetting but its also not working. but I have my changes saves so i am thinking to make new PR, is that alright with you ? I would take me a day.

[gkrivor]

It is because of merging a master branch. You can try to make a rebasing your branch locally to a current master. It should cleanup common changes and you will see only your changes.
I'm ok with new PR too :)

[RitikaxShakya]

It is because of merging a master branch. You can try to make a rebasing your branch locally to a current master. It should cleanup common changes and you will see only your changes. I'm ok with new PR too :)

Oh okay, i tried this as well but it doesn't cleanup but i will try again, if it works cool, if not then will open a new pr. Thank you!

[RitikaxShakya]

It is because of merging a master branch. You can try to make a rebasing your branch locally to a current master. It should cleanup common changes and you will see only your changes. I'm ok with new PR too :)

Hey I opened new PR Here, I have few questions, i have asked them in new PR.