Relocate badState helper to the top of ModelBuilder (#935)

As we've discussed in https://github.sec.samsung.net/STAR/nnfw/pull/809#discussion_r48662, once this helper was merged for easy to review, and then we've planed rearrange the order to easy to read in another PR.
This is why I make this PR.

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

Add the first draft of SW requirement specification (#920)

This commit adds the first draft of SW requirement specification.

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

Change externals/acl as a submodule (#827)

This patch removes externals/acl and adds ACL again as a submodule.

For issue #554

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

Resolve NN runtime unit test Assertion (#903)

Assertion generated in NN runtime unit test, need to check the need for assertion, disabling it as ToDo-NNRT

Add license to sources for tools, tests, include (#936)

Related issue: #840

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

Add license to sources under libs and src/support (#933)

Related issue: #840

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

Change TestValidation2.cpp to default NN Runtime test (#928)

This commit changes TestValidation2.cpp to default NN Runtime test.

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

Add test rpm for obs build (#914)

* Add test rpm for obs build

nnfw-test.rpm contains unittests and its install path
is /opt/usr/nnfw-test/

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

Reduce indent level (#927)

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

Enable rootfs using tools/cross/build-rootfs.sh (#929)

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

Add ANEURALNETWORKS_BAD_DATA check in validateOperandType/setFromPointer (#916)

* add additional ANEURALNETWORKS_BAD_DATA check in validateOperandType/setFromPointer

This commit adds additional ANEURALNETWORKS_BAD_DATA check in below
- validateOperandType
- setFromPointer

These codes have copied from android NN master branch(af63a36)

Fix : #805

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Add detailed comment to TODO comment

Add LICENSE in root directory (#913)

The purpose of LICENSE is to provide full text of license
that the corresponding source contains only partial and/or
link to full text.

It contains the following licenses:

- Apache License 2.0 (for ours)
- Mozilla Public License 2.0 (almost all of eigen)

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

Fix and update docs tizen xu4 file (#918)

This will fix xu4_tizen.md file that had a wrong script file name
- also update to latest tizen image file name that now doesn't have version number like `4.0`

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

Revert "Add assertion to check divide by zero (#892)" (#924)

This reverts commit 8abe4dba8ff6b5209561dc728b980fc7e8def816.

This was made for fixing the Coverity warning (#891) but it does not work.
We decide this warning to be a false alarm.

Revert "Disable memory bound check in MemoryTracker for 32bit arch" (#922)

This was made for fixing the UNREACHABLE_CODE detected by SVACE but it doesn't work.
We decide this warning to be a false alarm.

This reverts commit 3b01060eda993c1c9863237af561b04ed30c7dd0.
This reverts commit d22e4225050092a53b685f73cb0bcabc7c144a74.

Add doc for aarch64 build (#917)

* Add doc for aarch64 build

Add doc for aarch64 build. This is copied from doc for arm.

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

Add NNFW_KERNELS_ env on benchmark test (#912)

Adds NNFW_KERNELS env such as convFloat32, averagePoolFloat32 on
benchmark test. The envs are written on nnfw_kernel_env_list.txt.

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

Add additional ANEURALNETWORKS_BAD_DATA check into updateDimensionInfo (#910)

This commit adds additional ANEURALNETWORKS_BAD_DATA check into updateDimensionInfo.
These codes have copied from android NN master branch(af63a36)

Fix : #805

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

Can pass SVACE binary path (#900)

Can pass SVACE binary path using environment variable

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

Update NN Runtime TestTrivialModel test (#907)

This commit updates NN Rumtime TestTrivialModel test to android NN master branch.

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

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