Opinion about NNStreamer Android guide

Lee-WonJun commented 5 years ago

Android GUI NNStreamer Application Examples

How to build Android applications on Ubuntu

위의 가이드를 진행해본결과 개선의 여지가 필요하다고 생각되었습니다.

NNStreamer API Library for Android 에서 권한 문제
Build Android API

Run the build script in NNStreamer. After building the Android API, you can find the library(.aar) in $NNSTREAMER_ROOT/android_lib.

$ cd $NNSTREAMER_ROOT
$ bash ./api/android/build-android-lib.sh

해당 부분을 진행할떄 ./gradlew 파일의 퍼미션 문제로 fail이 발생합니다. 생성하는 build-android-lib.sh 가 생성하는 ./gradlew 파일의 퍼미션을 변경하는 sudo chmod +x ./gradlew 의 코드가 필요하다고 생각됩니다.

# Build Android library.
sudo chmod +x ./gradlew    #Add
./gradlew api:build

NNStreamer API Library for Android Prerequisite 관련 Android SDK Build-Tools 28.0.3 을 필요합니다. 위와 같은 부분을 진행할떄 Android SDK Build-Tools 28.0.3 설치를 요구합니다 Prerequisite 에 추가되면 좋겠습니다.
SVN 관련 해당 가이드들을 진행할때 파일설치시 SVN을 사용합니다. 파일을 받아올때 SVN 말고 다른 방식으로 guild 및 스크립트가 작성되있으면 좋겠습니다.
How to build Android applications on Ubuntu 가이드 관련 해당 가이드를 보고 예제및 api빌드하기가 어렵습니다. 수정되거나 NNStreamer API Library for Android,Android GUI NNStreamer Application Examples 등의 링크를 첨부하는 식으로 개선할수있을듯 합니다,
Android GUI NNStreamer Application Examples 에서 common 폴더 관련

해당 가이드의

Extract external libraries into common directory.

extfiles.tar.xz includes external library such as 'ahc'.

tensorflow-lite-1.13.tar.xz includes the libraries and header files of tensorflow-lite.

$ cd $ANDROID_DEV_ROOT/workspace/nnstreamer-example/android/example_app/common/jni
$ tar xJf ./extfiles.tar.xz
$ svn --force export https://github.com/nnsuite/nnstreamer-android-resource/trunk/android_api/ext-files/tensorflow-lite-1.13.tar.xz
$ tar xJf ./tensorflow-lite-1.13.tar.xz # Check tensorflow-lite version and extract prebuilt library
$ ls ahc tensorflow-lite

common 폴더까지 들어가서 해당부분의 extfiles 는 포함된 압축파일을 직접 풀고 tensorflow-lite 는 svn으로 받은뒤 압축을 푸는 과정을 거칩니다. 일관성이 맞지않고 자동화 스크립트를 만들어주어 개선할수 있을것 같습니다.

wooksong commented 5 years ago

@Lee-WonJun chmod할 때 sudo 필요없습니다.

Lee-WonJun commented 5 years ago

@wooksong 아 네 혹시몰라 sudo 사용했는데 없어도 되는군요

wooksong / contributon2019-nns

Opinion about NNStreamer Android guide #9

Build Android API