-
Notifications
You must be signed in to change notification settings - Fork 29
Android AICamera Demo #61
Conversation
…cause image recognizer not working
Xreki
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我试了下效果:
- face能检测到,前置摄像头和后置摄像头都能检测到
- pascal检测不到,摄像头对着一个目标很久也不出检测框,感觉应该不是速度的问题。这里可能需要检查一下
另外,可否加个小需求,就是显示一下每帧图片的处理时间?
Demo/Android/AICamera/README.md
Outdated
| @@ -0,0 +1,86 @@ | |||
| # PDCamera Android Demo with SSD Model | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Android Demo的README参考一下iOS的中文README.cn.md ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好,你那个README写得很好
Demo/Android/AICamera/README.md
Outdated
|
|
||
| ### QR code link | ||
|
|
||
| <img src="assets/qr_code_ios.png" width = "20%" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
更新一个Android Demo的二维码吧。我后面再来增加一个国内镜像的。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好,最后才加
Demo/Android/AICamera/README.md
Outdated
|
|
||
| ## Integrate Paddle C Library to Android | ||
|
|
||
| -Follow this guide [Build PaddlePaddle for Android](https://github.com/PaddlePaddle/Paddle/blob/develop/doc/mobile/cross_compiling_for_android_en.md) to generate paddle libs(include, lib, third_party). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个Markdown格式不正确。Mac上可以下载一个软件MacDown 来编译Markdown文件。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
恩
| #include <android/log.h> | ||
| #include <jni.h> | ||
| #include <memory> | ||
| #include <string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我这边直接build会报错,报错内容大概是找不到memcpy。将这一行改成#include <string.h>可以build。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好
| @@ -0,0 +1,61 @@ | |||
| package com.paddlepaddle.aicamera; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
package名也改成com.paddlepaddle.pdcamera?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好
| private static final String TAG = "ImageRecognizer"; | ||
| private static float[] means = {104, 117, 124}; | ||
|
|
||
| private static final String[] LABELS = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LABELS是不是作为参数传进来比较合适?因为这个ImageRecognizer类是支持多个不同的模型的。
还有means,这两个可能保存在SSDModel里面比较合适。
不过这样一来的话,似乎结果的解析也需要放到SSDModel里面去。。。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好的,MEANS 和 LABELS是应该放进SSDModel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
结果的解析可以留在ImageRecognizer吧,如果以后有不同模型,就把模型类型传进去再用对应的解析方法
…elf) to fix issues after change model, update README
Xreki
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我试过了,效果很赞 👍
再修缮下README,这个pr就可以merge了:
- 有些Markdown语法不对的地方
- 上传一下QR和显示界面的截图
Demo/Android/AICamera/README.md
Outdated
| - [Intro](#Intro) | ||
| - [Models](#Models) | ||
| - [Demo](#Demo) | ||
| - [Quick Install](#Quick Install) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be [Quick Install](#quick-install)
Xreki
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great. 我们可以先merge这个PR,后面再来修缮和添加中文文档。
No description provided.