Skip to content

[vivante] Add FP32 output conversion support for Vivante backend #20

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[vivante] Add FP32 output conversion support for Vivante backend #20

wants to merge 1 commit into from

Conversation

@anyj0527
Copy link
Member

  • Support "OutputType=FP32" or "OutputType=FLOAT32" custom option
  • This will convert ALL output tensors into float32 type.

Discussion / TODO

  • Support other output types.
  • Setting different type for each output tensor.
  • Modify the graph directly to produce given output type.

@anyj0527 anyj0527 requested a review from a team as a code owner August 19, 2025 04:59
@anyj0527 anyj0527 force-pushed the vivante-convert-output-fp32 branch from dbec3b9 to 401459a Compare August 19, 2025 05:01
jaeyun-jung
jaeyun-jung reviewed Aug 19, 2025
View reviewed changes
jaeyun-jung
jaeyun-jung reviewed Aug 19, 2025
View reviewed changes
@anyj0527 anyj0527 force-pushed the vivante-convert-output-fp32 branch from 401459a to c5c5bc8 Compare August 19, 2025 05:50
@anyj0527
[vivante] Add FP32 output conversion support for Vivante backend
f58286a 
- Support "OutputType=FP32" custom option
- This will convert ALL output tensors into float32 type.

Signed-off-by: Yongjoo Ahn <[email protected]>
@anyj0527 anyj0527 force-pushed the vivante-convert-output-fp32 branch from c5c5bc8 to f58286a Compare August 26, 2025 04:27
songgot
songgot approved these changes Aug 26, 2025
View reviewed changes
Copy link

@songgot songgot left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@songgot songgot songgot approved these changes

@jaeyun-jung jaeyun-jung jaeyun-jung approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@anyj0527 @songgot @jaeyun-jung