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/onnx-subgraph] Add test model downloading scripts #14690

Merged
merged 3 commits into from
Feb 19, 2025

Conversation

chenyx113
Copy link
Contributor

related issue: #14534
historical PR:
#14654
#14613

Add test onnx models downloading scripts for future smoking test, downloading will happen in installing path, will not effect git status.

ONE-DCO-1.0-Signed-off-by: Youxin Chen yx113.chen@samsung.com

@chenyx113 chenyx113 marked this pull request as ready for review February 18, 2025 06:18
@seanshpark
Copy link
Contributor

IMHO, it would be better to use [tools/onnx-subgraph] or [onnx-subgraph] for title prefix.

@chenyx113
Copy link
Contributor Author

IMHO, it would be better to use [tools/onnx-subgraph] or [onnx-subgraph] for title prefix.

ok, I will update as your comment after a while, thank you :)

Add test onnx models downloading scripts for future smoking test

ONE-DCO-1.0-Signed-off-by: Youxin Chen <yx113.chen@samsung.com>
modify the indent issue, remove useless code, add shebang in bash script,
modify the typing mistake

ONE-DCO-1.0-Signed-off-by: Youxin Chen <yx113.chen@samsung.com>
@chenyx113 chenyx113 changed the title [tools] Add test model downloading scripts [tools/onnx-subgraph] Add test model downloading scripts Feb 18, 2025
@chenyx113
Copy link
Contributor Author

@seanshpark, thanks for your detail comments, I have updated as review, please help check

remove the pip install in the bash script

ONE-DCO-1.0-Signed-off-by: Youxin Chen <yx113.chen@samsung.com>
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 thank you!

@seanshpark seanshpark merged commit 4bb312a into Samsung:master Feb 19, 2025
5 checks passed
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