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

model_profile provided with extra argument in __main__.py #33

Open
haojin2 opened this issue Jul 5, 2023 · 2 comments
Open

model_profile provided with extra argument in __main__.py #33

haojin2 opened this issue Jul 5, 2023 · 2 comments
Assignees
@ThanatosShinji
Labels
bug Something isn't working

Comments

@haojin2
Copy link

haojin2 commented Jul 5, 2023

Describe the bug
https://github.com/ThanatosShinji/onnx-tool/blob/main/onnx_tool/__main__.py#L93

To Reproduce
Just run a model profile with the main.py like: python3 -m onnx_tool -m profile -i model.onnx
@ThanatosShinji
Copy link
Owner

Do you mean need more args for model profile using from console?

@ThanatosShinji ThanatosShinji added the enhancement New feature or request label Jul 6, 2023
@haojin2
Copy link
Author

haojin2 commented Aug 10, 2023

No, you need to remove , dump_outputs=args.names to make it work, otherwise you're providing an extra argument into that function call

@ThanatosShinji ThanatosShinji added the bug Something isn't working label Aug 10, 2023
@ThanatosShinji ThanatosShinji self-assigned this Aug 10, 2023
@ThanatosShinji ThanatosShinji removed the enhancement New feature or request label Aug 10, 2023
ThanatosShinji added a commit that referenced this issue Aug 10, 2023
@ThanatosShinji
fix of #33
641247c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThanatosShinji ThanatosShinji

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@haojin2 @ThanatosShinji