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

[tools/model_explorer_circle] Load circle model with circle schema #14325

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

batcheu
Copy link
Contributor

@batcheu batcheu commented Nov 20, 2024

It will load circle model with circle schema which is generated by flatbuffer compiler.
The loaded model will be used for create model explorer graph in next PRs.

FYI, following command is used to generate circle schema.

flatc -p --gen-onefile --gen-object-api circle_schema.fbs

(Note: --gen-onefile option is required the latest version of flatc. (used version = 24.3.25))


issue: #14307
draft: #14318

It will load circle model with circle schema which is generated by
flatbuffer compiler.
In the near future, the schema will be auto-generated with the latest
instead of current builtin one.

ONE-DCO-1.0-Signed-off-by: Jonghwa Lee <[email protected]>
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

@seanshpark
Copy link
Contributor

It would be nice to have some descriptions in the README file how to generate schema file.

@batcheu
Copy link
Contributor Author

batcheu commented Nov 20, 2024

It would be nice to have some descriptions in the README file how to generate schema file.

Okay I'll update it in another PR.

@seanshpark seanshpark merged commit 9eaa697 into Samsung:master Nov 20, 2024
3 checks passed
@batcheu batcheu deleted the me_adapter/load_circle branch November 21, 2024 04:28
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.

2 participants