Skip to content

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