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

[PaddlePaddle Hackathon 2] task 62 PP-TinyPose Demo #250

Closed
wants to merge 1 commit into from

Conversation

unseenme
Copy link

No description provided.

在 Android 上实现实时的Pose关键点的检测功能


## 如何运行 Demo
Copy link
Collaborator

Choose a reason for hiding this comment

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

这块内容辛苦补充下,并展示运行效果示意图

'dest': 'OpenCV'
],
[
'src' : 'https://paddlelite-demo.bj.bcebos.com/models/ultra_light_fast_generic_face_detector_1mb_640_for_cpu_v2_10_rc.tar.gz',
Copy link
Collaborator

Choose a reason for hiding this comment

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

这些模型有使用吗?么有话,可以删除

${OpenCV_LIBS}
GLESv2
EGL
${log-lib})
Copy link
Collaborator

Choose a reason for hiding this comment

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

加个空行

# CMake builds them for you. Gradle automatically packages shared libraries with
# your APK.

set(PaddleLite_DIR "${CMAKE_CURRENT_SOURCE_DIR}/../../../PaddleLite-2.11-rc")
Copy link
Collaborator

Choose a reason for hiding this comment

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

建议目录名由PaddleLite-2.11-rc 改为PaddleLite

if (tensor_name == "image") {
auto inputTensor = predictor_->GetInputByName(tensor_name);
// Set the data of input image
// auto inputTensor = predictor_->GetInput(0);
Copy link
Collaborator

Choose a reason for hiding this comment

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

没有用的注释的语句辛苦删除

fontThickness);
}

// void Pipeline::Action_Process(cv::Mat *rgbaImage,
Copy link
Collaborator

Choose a reason for hiding this comment

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

没有用的注释,辛苦删除下

VisualizeKptsResults(results, results_kpts, &rgbaImage, false);

// Visualize the status(performance data) to the origin image
// VisualizeStatus(readGLFBOTime, writeGLTextureTime, preprocessTime+preprocessTime_kpts,
Copy link
Collaborator

Choose a reason for hiding this comment

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

同上

double postprocessTime, cv::Mat *rgbaImage,
std::vector<RESULT_KEYPOINT> &results_kpts,
std::vector<RESULT> &results);
// void Action_Process(cv::Mat *rgbaImage,
Copy link
Collaborator

Choose a reason for hiding this comment

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

同上

}

void NHWC3ToNC3HW_bn(const float *src, float *dst, const float *mean,
const float *std, int width, int height) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个函数可以和NHWC3ToNC3HW 合并,如果没有归一化处理,mean 是0,scale 是1 就行
(din - mean) / scale

*(dst_c1++) = *(src++);
*(dst_c2++) = *(src++);
}
// for (; i < size; i++) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

无用的注释可以删除

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

另外,你的代码格式没有通过检测
image
建议安装 pre-commit,例如:pip install pre-commit 然后:pre-commit run -a

Copy link
Collaborator

Choose a reason for hiding this comment

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

另外,readme 文档格式请参考已有的readme 文档,辛苦重新更新下

@unseenme
Copy link
Author

unseenme commented May 5, 2022

@chenjiaoAngel 感谢评审。后面会一一修改。

@paddle-bot paddle-bot bot closed this Mar 5, 2024
Copy link

paddle-bot bot commented Mar 5, 2024

Automatically closed by Paddle-bot.

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