Skip to content
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

[luci/service] Support shape inference for non-const paddings #13949

Merged
merged 5 commits into from
Sep 11, 2024

Conversation

icodo98
Copy link
Contributor

@icodo98 icodo98 commented Sep 8, 2024

This PR supports infer shape of pad operation with dynamic paddings.
ONE-DCO-1.0-Signed-off-by: JuYoung Lee [email protected]

@icodo98 icodo98 added PR/ready for review It is ready to review. Please review it. SSAFY labels Sep 8, 2024
@icodo98 icodo98 requested review from a team September 8, 2024 08:59
This PR supports infer shape of pad operation with dynamic paddings.

ONE-DCO-1.0-Signed-off-by: JuYoung Lee [email protected]
@icodo98
Copy link
Contributor Author

icodo98 commented Sep 8, 2024

related to : #13790
draft : #13877

fix typo of annotation and adjust lines withs suggestions.

ONE-DCO-1.0-Signed-off-by: JuYoung Lee [email protected]
@icodo98 icodo98 changed the title [luci/service] support non-const paddings [luci/service] Support shape inference for non-const paddings Sep 9, 2024
Change TC to have direct relation with changed code.

ONE-DCO-1.0-Signed-off-by: JuYoung Lee [email protected]
Copy link
Contributor

@shs-park shs-park left a comment

Choose a reason for hiding this comment

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

I left some questions and comments.
PTAL.
=)

compiler/luci/service/src/Nodes/CirclePad.test.cpp Outdated Show resolved Hide resolved
compiler/luci/service/src/Nodes/CirclePad.test.cpp Outdated Show resolved Hide resolved
compiler/luci/service/src/Nodes/CirclePad.test.cpp Outdated Show resolved Hide resolved
icodo98 and others added 2 commits September 10, 2024 17:05
remove it and discuss about it in another issue.

ONE-DCO-1.0-Signed-off-by: JuYoung Lee [email protected]
remove unnecessary line.

Co-authored-by: SaeHie Park <[email protected]>
Copy link
Contributor

@shs-park shs-park left a comment

Choose a reason for hiding this comment

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

LGTM!
=)

Let's create another issue to discuss additional NEG tests.

Copy link
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

Copy link
Contributor

@zetwhite zetwhite left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit 0f8808e into Samsung:master Sep 11, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/ready for review It is ready to review. Please review it. SSAFY
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants