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.

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

[onnx-tools] Show total number of nodes #14303

Merged
merged 1 commit into from
Nov 5, 2024
Merged

[onnx-tools] Show total number of nodes #14303

merged 1 commit into from
Nov 5, 2024

Conversation

seanshpark
Copy link
Contributor

This will revise onnx-dump to show total number of nodes.

@seanshpark
[onnx-tools] Show total number of nodes
de7bd22 
This will revise onnx-dump to show total number of nodes.

Signed-off-by: SaeHie Park <[email protected]>
@seanshpark
Copy link
Contributor Author

this will show something like this

[Operators] ---------------------------
 1098 Constant
  524 Add
   36 Conv
   30 Gather
...
   30 Erf
    2 Resize
----- -----
 4661 Total

[Initializers] ------------------------

@seanshpark seanshpark requested a review from a team November 5, 2024 07:48
@seanshpark
Copy link
Contributor Author

cc @ejjeong

jinevening
jinevening approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@jinevening jinevening 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 seanshpark merged commit 75f98dd into Samsung:master Nov 5, 2024
8 checks passed
@seanshpark seanshpark deleted the onnxto_nodes_total branch November 5, 2024 21:14
BLee-bot pushed a commit to BLee-bot/ONE that referenced this pull request Nov 6, 2024
@seanshpark @BLee-bot
[onnx-tools] Show total number of nodes (Samsung#14303)
36bf014 
This will revise onnx-dump to show total number of nodes.

Signed-off-by: SaeHie Park <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jinevening jinevening jinevening 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.
2 participants
@seanshpark @jinevening