Skip to content

[ARM]Add profile script#10064

Merged
wz1qqx merged 6 commits intoPaddlePaddle:developfrom
wz1qqx:add-profile-script
Mar 10, 2023
Merged

[ARM]Add profile script#10064
wz1qqx merged 6 commits intoPaddlePaddle:developfrom
wz1qqx:add-profile-script

Conversation

@wz1qqx
Copy link
Collaborator

@wz1qqx wz1qqx commented Mar 6, 2023

PR devices

Arm

PR types

New features

PR changes

Docs

Description

add profile.sh under lite/tools, It supports one-click conversion of the model into an nb model and transfers it to the device for time profile testing, and finally saves the results in the specified directory to improve daily work efficiency

@paddle-bot
Copy link

paddle-bot bot commented Mar 6, 2023

Thanks for your contribution!

@@ -0,0 +1,13 @@
AlexNet;images:1,3,224,224
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是否以常量或字符串参数放在profile.sh里面?不要单独以文件形式存在。

# can be customed from terminal input
MODELS_DIR=test
OPT_DIR=nb_models
ANDROID_DIR=/data/local/tmp/profile
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是否支持 armlinux?建议后续参考

function ssh_device_check() {
同时支持 ssh 和 adb

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

当前仅支持adb方式,目的是让普通用户快速测试模型性能,用户和rd间通过profile输出文档进行模型性能问题的沟通。后续发版再添加armlinux支持。

Copy link
Collaborator

@hong19860320 hong19860320 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wz1qqx wz1qqx merged commit 5ef1258 into PaddlePaddle:develop Mar 10, 2023
@wz1qqx wz1qqx deleted the add-profile-script branch March 10, 2023 06:03
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