Fix build_rootfs.sh message (#909)

This will fix build_rootfs.sh script that shows `[: missing `]'` warning

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

nnapi_test check arguments (#902)

This commit fixes when .tflite file is not given from the argument.
Which has caused segmentation fault.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Enable build aarch64 for tizen&cross (#885)

Enable build aarch64 for tizen&cross

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Add license for sources under src/runtime (#904)

Related issue: #840, #839

Signed-off-by: Sanggyu Lee <sg5.lee@samsung.com>

Disable memory bound check in MemoryTracker for 32bit arch (#886)

* Disable memory bound check in MemoryTracker for 32bit arch

This commit disables memory bound check in MemoryTracker for 32bit archtecture.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Change supported bit check mechanism

Add assertion to check divide by zero (#892)

- Add assertion to check divide by zero
  In this context, the value will not be zero.
  But coverity complains this, so let's guard it with assertion.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Free memory at early exit (#899)

- Free memory at early exit
  This was reported by Coverity (118950 Resource leak)

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Introduce badstate checker in ModelBuilder (#809)

* Introduce badstate checker in ModelBuilder

This commit introduces badstate checker in ModuleBuilder to fix #804 failure.
These codes have copied from android NN master branch(af63a36).

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Move badState to the end of file

tflite file generator (from Tensoflow model in python source code) (#810)

PR for #660

Summary of current work.
1. One monolithic file was splitted into several py files.
   - Base class (a template for subclasses)
   - Several subclass which define model (to be converted later)
   - main()
1. Code to generate *.tflite was added (by calling TOCO internally)
   (Tensorflow model in python code is converted to *.tflite file.)

Remaining work:
- Supporting multiple outputs

Fix coverity error (118949) (#897)

Fix coverity error (118949) (#897)

Exit on nnapi_test fail (#890)

1. remove `set +e` from nnapi_test

Now, test fails on any nnapi_test fail.

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Add license for sources under src/kernel (#879)

Related issue: #840, #839

Signed-off-by: Sanggyu Lee <sg5.lee@samsung.com>

Add a script for generation RootFS for Android (#884)

This script generates a standalone build environment for Android.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Check if ashmem region is created. (#878)

This commit checks if ashmem region is created.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Workaround: temporary disable concat2 test (#889)

Temporary disable concat2 test until problem solved

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Build runtime only for SVACE (#888)

Build runtime only for SVACE report

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove execute mode from source files in NN runtime (#876)

- Remove execute mode from source files in NN runtime

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Remove dead code in Activation.cpp (#862)

- Remove dead code in Activation.cpp
- This was reported as UNCHECKED_FUNC_RES.STAT from SVACE

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Add rootfs for ARM64 (#877)

* Add rootfs for ARM64

Prepares nnfw to support cross/ARM64 build

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Remove double semicolon in Memory code (#881)

This commit removes double semicolon in Memory code.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Close 'dupfd' if returned unexpected null (#861)

This commit closes 'dupfd' if returned unexpected null.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Make option: runtime build only (#864)

Introduce new make option: runtime build only
Can build runtime only by "make runtime"

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Disable depthwiseConvQuant8 in NN runtime (#866)

- Disable depthwiseConvQuant8 in NN runtime
- We disable this because
  - we don't have test set for this yet
  - and we want to exclude unused code from code coverage test.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Update pthread to `${LIB_PTHREAD}` (#868)

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

script to download model files (#858)

Usage : ./run_test.sh --download=on

Fix incorrect md5sum for inceptionv3_module (#854)

[Build] Support Android build for nnapi_test (#869)

Support Android build for nnapi_test and its dependencies.
Need to add a document how to create rootfs.

Fix #515

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Fix typo of a condition (#873)

`&` here is supposed to be `&&`.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Fix memory leak in Conv2D.cpp (#865)

- Fix memory leak in Conv2D.cpp
- These were reported as MEMORY_LEAK.EX in SVACE (Issue #849, #850)

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Remove unnecessary return statement (#846)

- Remove unnecessary return statement which make below code unreachable.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Remove tools and depend directory from coverage (#856)

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Fix c++ and python coding format in tools (#843)

Fix c++ and python format based on decided coding rule

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

upload concatenation files to the mooo server (#836)

Signed-off-by: SungJin Choi <lotieye.choi@samsung.com>

Docker build for svace analyzer (#844)

Add script for docker build with svace analyzer

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Fix test error on tizen (#847)

dirname does not accept multiple arguments.
To run test on tizen, dirname is executed as a part of find command.

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Framework tests for RELU and RELU6 operations (#830)

* Framework tests for RELU and RELU6 operations

This commit enables the framework tests for RELU and RELU6 activation functions

* Update config.sh

Add reshape op to NNFWKernel to test performance (#835)

Signed-off-by: SungJin Choi <lotieye.choi@samsung.com>

Remove external/protobuf and tools/tf_model_tools (#826)

For #553

This patch removes external/protobuf and tools/tf_model_tools.

tools/tf_model_tools has been copied to
https://github.sec.samsung.net/RS7-RuntimeNTools/nnas_tools

Signed-off-by: Sangmin Seo <sangmin7.seo@samsung.com>

TFLite model tool: PEP8 Format (#821)

Fix python code format based on PEP8

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove local repo from gbs build (#817)

1. remove local repo
2. add --include-all option to gbs build command for local test

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Check C++ files: .cc extension (#822)

Add .cc extension to check C++ coding rule

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Introduce 'libs' directory (#820)

This commit introduces 'libs' directory which will hold various
libraries used for runtime implementation.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Fix build break for benchmark on ci (#819)

Print json with quot. Exactly its spec is not well descriptive. This
solution is workaround.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Update document of docker image (#790)

- Update document of docker image
- The latest version of docker image is nnfw_docker:1.4

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[Build] Fix hardcoded value from Makefile (#786)

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Print benchmark's result to json (#811)

* Print benchmark's result to json

Print benchmark's result to json for visualizing on jenkins. The json
file's content fit to specification of benchmark(one of jenkins plugin).

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Unify NEON and OpenCL versions of Reshape unit test (#813)

- extract common features to `src/Reshape.test.h`
- implement different parts into `neon/Reshape.test.cpp` and
  `cl/Reshape.test.cpp`

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Add coverage report script for CI (#801)

This script is used on CI to generate lcov coverage info.

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Check nullptr of the buffer in setInput (#807)

* Check nullptr of the buffer in setInput

This commit checks nullptr of the buffer in setInput to fix #806.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Separate nullptr checks in setInput

Enabling RELU and RELU6 (#787)

Enable relu and relu6 operations in cpuexecutor

Enable depthconv and reshape tests in config (#788)

Enable the depthwiseconv and reshape in test config files for the operations to be tested in test_run

Separate ACL build type output folder (#803)

This will use ACL build option `build_dir` to separate build type (debug,release) to have its own folder

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Clean up Utils in common (#799)

This commit cleans up Utils in common.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Build tests only when googletest is built (#802)

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Unify NEON and OpenCL implementation of reshape layer (#800)

NEON and OpenCL implementations of reshape layer are very similar.
This commit
- extracts common features to `src/Reshape.h`
- implement different parts into `neon/Reshape.cpp` and `cl/Reshape.cpp`.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

[NNOP] Unify NEON and OpenCL implementaion and tests (#798)

This commit unifies NEON and OpenCL implementation and tests.
- common statements are extracted to `acl/src/FullyConnected.h` and
  `acl/src/FullyConnected.test.h`
- Different parts are described in `cl` and `neon` directories.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Separate acl build and install step (#785)

This will separate acl library build and install step in Makefile
- this is needed for only want to copy libraries after make clean
- there is no change for 'make acl'

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Change realpath to readlink -f on test driver (#784)

Instead of realpath, use readlink -f because there
is no realpath on taos.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Introduce NEUniqueTensor (#779)

NEUniqueTensor is a tensor wrapper for arm_compute::Tensor.
This is similar to CLUniqueTensor.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Remove wget verbose in tizen cross build (#782)

Remove wget verbose in tizen cross build script

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Enable Depthwiseconv2D of CpuExecutor (#617)

Add the DEPTHWISECONV2D operation file and enable the operation with float support.

Fix test driver not to print message on benchmark (#770)

Fix test driver not to print message such as ones from wget on benchmark
mode.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

[Build] Add arch `arm64` in Makefile (#775)

Add archicture `arm64`.
For arm64, android is the only OS supported for now.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Enable acl/neon backend (#776)

This will add acl/neon kernel implementations to kernel lookup tables

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

TFLite Model tool: Use argparse library (#772)

Use argparse library to define and parse argument

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

TFLIte model parser: More tensor information (#732)

* TFLIte model parser: More tensor information

Print tensor debugging name
Print shape as scalar if data rank is 0

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Update README.md

Update README.md to print tensor name

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

TFLite run: Define more custom operator (#777)

Define more custom operator using updated model
Print only.

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Clean include section of ExecutionBuilder (#778)

This commit cleans include section of ExecutionBuilder.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Update TestValidation.cpp to android master branch for runtime (#742)

* Update TestValidation.cpp to android master branch for runtime

This commit updates TestValidation.cpp to anroid master branch(5596ca500) for runtime.
Related : #611

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Change master test to TestValidation2.cpp

Use const for readonly shape (#773)

This will add const reference to method that return value is used read only
- this will prevent memory copy

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Introduce 'make_alloc' helper (#769)

This commit introduces 'make_alloc' helper which makes it easy to use
malloc on a specific type, and revises random test implementations using
this new helper.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[CMake] Do not force TARGET_OS to be HOST_OS (#771)

Currently TARGET_OS was always forced to be HOST_OS.
It have not been an issue since all OS we handle was linux, so far.
For the future, we should TARGET_OS is set to HOST_OS only if it is not given.

Register concat to nnfw kernel table (#767)

This will add concatenationFloat32 on ACL to kernel lookup table

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Fix wrong report dir on test_driver.sh (#754)

When test_driver.sh run unittest, the report dir was changed by the
path where test_driver.sh run. This commit fixes the report dir.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Add format check package to docker image (#764)

Add clang-format-3.9, pip, and yapf for format checker

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Add svace to gitignore (#766)

This will add SVACE data folder .svace-dir to .gitignore

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

[CMake] Extract pthread as a variable (#760)

Instead of just writing `pthread`, use `${LIB_PTHREAD}`.
This is for the future, so Android config will override it to an empty value.
Since BIONIC libc library contains pthread library so it will emit an error if
`-lpthread` is passed to linker.

Update tflite file API (#711)

Update tflite file API generated by flatc
Using update schema file in tensorflow 1.7

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

resolve #757 (#765)

Signed-off-by: SungJin Choi <lotieye.choi@samsung.com>

Add lcov-to-cobertura conversion tool (#755)

* Add lcov-to-cobertura conversion tool

This script converts lcov result to covertura format.
This script will be used on CI side for publishing line coverage result.

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>
* Add original source link

Show max absolute/relative diff (#761)

This commit revises NNAPI test framework to show max absolute/relative
diff information at the end.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Stringfy COVERAGE_BUILD (#762)

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Add a unit test of FC-FC (#759)

This commit adds a unit test of an FC layer followed by another FC
layer.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Add test coverage build (#744)

* Add test coverage build

1. add flags for test coverage build
2. add script to enable on CI

* Add OPTION init

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Add lcov to docker image (#752)

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Fix wrong running benchmark test (#753)

Fixes wrong benchmark test. According to introducing run_test.sh's
options, it needs to add options properly.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Change input/weights/bias/output shape of FC according to NNAPI spec (#750)

This commit changes input/weights/bias/output shape
to follow the NNAPI specification.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Revise code of printing summary of framework test on test_driver.sh (#748)

Revises code of printing summary of framework test on test_driver.sh by
applying options from run_test.sh.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Add summary of verification on test driver (#749)

Enables test driver to print summary(tap) of the result on verification

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Initial code of Concatenation on ACL (#746)

This will add initial implementation of Concatenation on ACL CL and NEON

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Print summary of benchmark on test_driver.sh (#739)

Enables test_driver.sh to print summary of benchmark. The printed
summary has only 'mean' result of benchmark.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Change assertion of MatrixWeightAccessor to accept 1D and 2D (#747)

This commit change assertion condition to accept both 1D and 2D tensors
for MatrixWeightAccessor similar to MatrixOutputAccessor.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Coding rule check script (#730)

Check coding rule for c++ code and python code
Use clang-format to check c++ coding rule, and use yapf to use python coding  rule based on PEP8

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Clang format file for cpp coding rule (#712)

Define C++ coding rule

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[NNOP] Introduce MatrixOutputAccessor (#741)

This commit introduces MatrixOutputAccessor to process 2D output tensors.

Signed-off-by: Junghyun Kim <jh0822.kim@samsung.com>

Register ACL operators to NN runtime (#721)

This will add ACL operators to NN runtime "acl" items so that we can use them
- Operation itself is not verified yet, we need to test and fix if problems exist

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

Rename and relocate logging for runtime (#713)

This commit renames and relocates logging for runtime (remove android dep.)

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Fix unusual assertion in CONV2D Op. (#707)

This commit fixes unusual assertion in CONV2D Op likes #430.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Implement Usage of test_driver.sh (#727)

Fills Usage function which prints help messages for options.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>

Fix softmax with ACL (#733)

* Fix softmax with ACL

This will fix softmaxFloat32 implementation to use rank 2 input
- this change is to run Inception V3 model
- as ACL shrinks rank as high order size is 1, input/output actually be a vector

Signed-off-by: SaeHie Park <saehie.park@samsung.com>
* add test for rank 4

* apply for non-support ranks

* return false for release build