Skip to content

Conversation

hseok-oh
Copy link
Contributor

@hseok-oh hseok-oh commented Aug 19, 2025

This commit supports internal output allocation on MultiModelExecutors.
This commit includes two multimodel shape inference tests and update to pass tests

  • External output buffer (default)
  • Internal output buffer allocation

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]


Draft: #15928

This commit supports internal output allocation on MultiModelExecutors.
This commit includes two multimodel shape inference tests and update to pass tests
- External output buffer (default)
- Internal output buffer allocation

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
@hseok-oh hseok-oh force-pushed the test_shapeinf_multimodel branch from 8ae5466 to 492d838 Compare August 20, 2025 04:54
@hseok-oh hseok-oh changed the title [onert] Add multimodel shape inference test [onert] Support internal output allocation on MultiModelExecutors Aug 20, 2025
@hseok-oh hseok-oh requested a review from a team August 20, 2025 04:55
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Aug 20, 2025
@hseok-oh hseok-oh marked this pull request as ready for review August 20, 2025 04:55
Copy link
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.

It seems to be the same issue as the suggestion in #15925. (sizeof, output_buffer check)

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

@hseok-oh hseok-oh merged commit b88f739 into Samsung:master Aug 21, 2025
10 checks passed
@hseok-oh hseok-oh deleted the test_shapeinf_multimodel branch August 21, 2025 08:29
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.

2 participants