Skip to content

[onert] Support constant input BatchMatMul#15033

Merged
hseok-oh merged 2 commits intoSamsung:masterfrom
hseok-oh:support_const_bmm
Apr 3, 2025
Merged

[onert] Support constant input BatchMatMul#15033
hseok-oh merged 2 commits intoSamsung:masterfrom
hseok-oh:support_const_bmm

Conversation

@hseok-oh
Copy link
Copy Markdown
Contributor

@hseok-oh hseok-oh commented Apr 2, 2025

This commit adds support for constant input BatchMatMul.
It includes BatchMatMul constant input tests.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh hseok82.oh@samsung.com


Draft: #14905
Related issue: #14951

@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Apr 2, 2025
@hseok-oh hseok-oh requested a review from a team April 2, 2025 06:07
This commit adds support for constant input BatchMatMul.
It includes BatchMatMul constant input tests.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
@hseok-oh hseok-oh force-pushed the support_const_bmm branch from 226957a to 7dd42b1 Compare April 2, 2025 06:12
Comment thread runtime/compute/cker/include/cker/operation/BatchMatMul.h Outdated
fix typo

Co-authored-by: SeungHui Youn <61981457+zetwhite@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
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 👍

Copy link
Copy Markdown
Contributor

@ys44kim ys44kim left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit 9b6b273 into Samsung:master Apr 3, 2025
10 checks passed
@hseok-oh hseok-oh deleted the support_const_bmm branch April 3, 2025 00:56
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants