Skip to content

Conversation

@anyj0527
Copy link
Member

  • Without .so file to intialize the given model, this supports json based initialization.
  • Fix README.md accordingly.

- Without .so file to intialize the given model, this supports json based initialization.
- Fix README.md accordingly.

Signed-off-by: Yongjoo Ahn <[email protected]>
@anyj0527 anyj0527 requested a review from a team as a code owner June 10, 2025 05:28
- Refactor and fix memory leaks

Signed-off-by: Yongjoo Ahn <[email protected]>
@anyj0527 anyj0527 force-pushed the vivante-support-json-info branch from e19c6c8 to cf1cd9d Compare June 10, 2025 05:57
Copy link

@again4you again4you left a comment

Choose a reason for hiding this comment

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

This approach is reasonable for me. 👍

anyj0527 added 2 commits June 11, 2025 12:45
- Get dtype values only when they are explictly given.

Signed-off-by: Yongjoo Ahn <[email protected]>
- `_NNS_FLOAT16` is always available. Use it.

Signed-off-by: Yongjoo Ahn <[email protected]>
Copy link

@abhajaswal abhajaswal left a comment

Choose a reason for hiding this comment

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

changes look fine.

@myungjoo
Copy link
Member

@anyj0527 With acknowledgment from users, you may proceed.

@abhajaswal
Copy link

Thanks alot for your efforts, I have confirmed the changes work well on Tizen 10.0 image for FHUb A311D
Checked both Yolov8 models & InceptionV3.

I have approved the PR.
I think we can merge the changes, We would further do memory leaks and other sanity tesing in parrallel.

@anyj0527
Copy link
Member Author

@abhajaswal Thanks for your thorough review

@anyj0527 anyj0527 merged commit ad70eb6 into nnstreamer:main Jun 12, 2025
5 checks passed
@anyj0527 anyj0527 deleted the vivante-support-json-info branch June 12, 2025 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants