Rename utils for NNAPI to internal converters (#4429)

Rename file name from TypeConvert.* to NNAPIConvert.*
Rename util function name of NNAPI frontend struct converters
Update comments

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

Revise frontend operand add API (#4427)

- Add log message for fail
- Introduce wrapper method

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

Introduce shape converter (#4428)

Introduce shape converter from NNAPI operand type to neurun shape

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

[neurun] Introduce Index::valid method (#4425)

Introduce `valid` method for Index class. True if initialized otherwise
False.

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

Apply IStage to IStageGenerator and PlanBuilder (#4414)

Apply IStage(AtomicStage) to IStageGenerator and PlanBuilder.
StageSequence will be applied for Subgraph soon.

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

Revise OperandUsage and related methods (#4408)

Rename OperandUsage as Usage
Remove setters for each enum value and use one setter
Rename getter and setter for usage

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

Remove tabs in libs/tflite/CMakeLists.txt (#4424)

We haven't to use tab but space instead.
This patch removes two tabs and uses spaces instead.

Signed-off-by: Inki Dae <inki.dae@samsung.com>

[PACL] Change getARMComputePermutationVector to make it generally (#4372)

This commit changes getARMComputePermutationVector to make it generally usable to be able to support NHWC.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Introduce pointer union to handle tensor data (#4421)

Introduce pointer union in cpu kernel to handle tensor data
It removes redundant reinterpet_cast

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

Use target_compile_definitions instead add_definitions (#4419)

* Use target_compile_definitions instead add_definitions

We moved TFLITE relevant flags to tflite directory so
this flag setting should affect only this target.

This patch replaces add_definitinos with target_compile_definitions
so that this setting can affect only local target.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Do not relocate if block

also fix a typo.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* split a line

Signed-off-by: Inki Dae <inki.dae@samsung.com>

Revise frontend model object creation and finish (#4412)

* Revise frontend model object creation and finish

- Add log message for fail
- Add noexcept keyword to wrapper method
- Move duplicate finish call check from wrapper to NNAPI implementation

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Return false if exception catched

Move TFLITE relevant flag setting to proper place (#4411)

TFLITE_MAJOR_VER and TFLITE_MINOR_VER flags are used to check
which version of TFLITE should be referred by NNFW at build time.
So these are related to only TFLITE relevant code.

This patch moves the setup script code of these two flags
from CMakeLists.txt to libs/tflite/CMakeLists.txt

Signed-off-by: Inki Dae <inki.dae@samsung.com>

[neurun] Remove missed skip lists of cpu test for SoftMax (#4417)

This commit removes missed skip lists of cpu test for `SoftMax`.

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

Introduce type info converter for frontend (#4418)

Introduce converter from NNAPI operand type to runtime internal type info

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

Support operator to compare TypeInfo (#4415)

Operator overloading to compare TypeInfo

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

Introduce .gitattributes for github/linguist (#4409)

This repo is shown as a Python project due to many test spec files.
This commit prevents searching in generated test directory from linguist
(repo language detection).

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

[neurun] Use 'GenericFullyConnectedLayer' (#4396)

* [neurun] Use 'GenericFullyConnectedLayer`

We introduced `GenericFullyConnectedLayer` to support 4d input.
This commit applies and uses `GenericFullyConnectedLayer` instead of `CLFullyConnectedLayer` directly.

and then we don't need to dealling with `4D input tensor` in `ConstantInitializer` for `FullyConnected` operation anymore, so removed it.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Implement calcOffset for cpu and add workaround

Remove using default backend for input and output (#4370)

Using default backend is meaningless because of input/output value setting optimization
Use frontend layout information by hardcording
Skip set input tensor value when its backend is not allocated

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

Modified for Tensorflow 1.12 (#4378)

* Modified for Tensorflow 1.12

Fixed error while converting pb to tflite by applying changes in Tensorflow 1.12.

Signed-off-by: Hyun Sik Yoon <hyunsik.yoon@samsung.com>
* Modified to a TF 1.12 term "Tensorflow Lite Converter"

[neurun] Support 'quant8' for some cpu kernels (#4402)

This commit supports `quant8` data type for some cpu kernels except below.

- `ConvolutionLayer` : no match result
- `FullyConnectedLayer` : need to revise to use `tflite`

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

[neurun] Do not correct dims for Concat operands (#4394)

Concat operands should not change dimensions for its inputs and outputs.

For example, rank 4 (8, 8, 2048, 1) is changed to rank 3 (8, 8, 2048).
It would break Concat operation.

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

Support NHWC to SpaceToDepthKernel (#4369)

This commit supports NHWC to SpaceToDepthKernel

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Introduce IStage, AtomicStage and CompositStage (#4374)

* Introduce IStage, AtomicStage and CompositStage

Introduce IStage, AtomicStage and CompositStage
- IStage: pure interface for replacing current Stage
- AtomicStage: Stage which can be called as atomic op
- CompositStage: A Stage or Stages which can be grouped

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>
* Move {Atomic,Composit}Stage to each own file

* Change struct to final class

* Change CompositStage to StageSequence

* final class to class final

Check tflite version in build time (#4371)

* Check tflite version in build time

This patch seperates what OBS_BUILD flag means into.
- OBS_BUILD flag decides where external packages should be
  installed from - if OBS_BUILD == 1 then from tizen package server
  otherwise from upstream.
- TFLITE_VER means what tensorflow-lite version nnfw uses - if
  TFLITE_VER == 190 then nnfw uses tensorflow-lite v1.9.0 for Tizen
  otherwise tensorflow-lite v1.12.0. In default TFLITE_VER is 1120.

For this, I added a TFLITE_VER flag, which will pass proper version
information - 190 for gbs build or 1120 in default - to code side.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Seperated TFLITE_VER into TFLITE_MAJOR_VER and TFLITE_MINOR_VER

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Updated nnapi_delegate.h I missed.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Updated and cleaned up

- Cleaned up CMakeLists.txt file
- Added TFLITE_MAJOR_VER and TFLITE_MINOR_VER flags when coverage_build == 1

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Updated description

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Fix typo

Signed-off-by: Inki Dae <inki.dae@samsung.com>

Generate compiler command information during build (#4382)

* Generate compiler command information during build

- Set CMAKE_EXPORT_COMPILE_COMMANDS option to generate
  `compile_commands.json` file while cmake build.
- It will be used to generate semantic information from various tools.
- Leave a comment about unsupported feature in CMake version 3.1.0

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

Introduce Subgraph as Node (#4373)

* Introduce Subgraph as Node

Introduce Subgraph as Node. This Subgraph would be handled as extrordinay
Node so that running such as group could be possible.

Signed-off-by: Yongseop Kim <yons.kim@samsung.com>
* Remove unnecessary header

* Change addOperation name and index of Element of Subgraph

Set default value of HOST_OS (#4384)

- Set default value of HOST_OS in platform identification script.
- For using cmake directly without make.

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

Add CUSTOM(Abs) op to nnapi_delegate (#4379)

This commit adds CUSTOM(Abs) op to nnapi_delegate.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Arrange GTests in tizen skip lists in order (#4093)

This patch arranges the same for better organizing and tracking

Signed-off-by: prasannar <prasanna.r@samsung.com>

[neurun] Use 'UNUSED_RELEASE' macro for unused variable (#4362)

* [neurun] Move and use 'NO_USE' macro

This commit moves and use `NO_USE` macro instead of the casting with `(void)`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* change macro name

Block cpu kernel build on gbs build (#4355)

Block cpu kernel build on gbs build because of build fail
Remove this if ready

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

Enable DepthwiseConv2D in neurun (#4319)

* Enable DepthwiseConv2D in neurun

- Enable implicit-padding depthwise conv2d in neurun

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>
* enable explicit padding

- enable explicit padding

* Fix incorrect comment

* Fix incorrect year on License Statement

* fix format by checker

[neurun] Add more comment on cpu::TensorBuilder (#4365)

Add supplementary explation for the reason why `allocate` does nothing.

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

Fix logging runtime bug (#4368)

Fix logging runtime bug: print address after console logging

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

Remove unnecessary kernels from ARMComputeEX (#4325)

This commit removes unnecessary kernels from ARMComputeEX

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Support NHWC to DepthToSpaceKernel (#4363)

* Support NHWC to DepthToSpaceKernel

This commit supports NHWC to DepthToSpaceKernel.

* Change a preprocessor argument of kernel for DepthToSpace

This commit changes a preprocessor argument of kernel for DepthToSpace to prevent misunderstandings.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Introduce operand::IndexMap (#4364)

* [neurun] Introduce operand::IndexMap

Introduce `operand::IndexMap` which is an alias of `unordered_map` with
key as `operand::Index`.

* Update ifdef guard format

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

[neurun] Remove asserts for SoftMax validation to support tensors (#4361)

Removed asserts for `SoftMax` in `PACL` to support tensors.
These asserts copied from `PACL` before changing.

This commit removes asserts for `SoftMax validation` to support tensors on `neurun` too.
And removes test skip list related `SoftMax`.

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

[neurun] Support explicit padding and activation for MaxPool2D (#4327)

We can pass all of generated `MaxPool2D` tests.

This commit supports explicit padding and activation for `MaxPool2D` to pass related tests.

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

[neurun] Fix reshape bug (Apply #4254) (#4333)

* [neurun] Fix reshape bug (Apply #4254)

Related : #2147, #4254

In #4254, It's set `apply_dim_correction` of `input`/`output` shapes to `false in `PACL`.
This commit applies #4254 to `neurun` and remove related skip test list.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Revise CLTensor constructor

[neurun] Remove test skip list that can be passed (#4359)

This commit removes tests skip list that can be passed after some PR are merged.

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

Build neurun only should be possible (#4357)

Building `nnfw` for ubuntu with `-DBUILD_NEURUN=1 -DBUILD_PURE_ARM_COMPUTE=0`
builds both of neurun and pacl. It should build neurun only as user's input parameters.

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

Remove non-const getter in Plan (#4351)

Remove non-const getter in Plan
Use operand context and operation sequence directly for each compile phase
Generate plan after compile finished

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

Rename GENERATE_FILES to GENERATE_NNAPI_TESTS (#4356)

To skip generating nnapi tests, `GENERATE_FILES=0` has been used.
This patch changes the flag's name to more specific one since
`GENERATE_FILES` are too general.

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

[neurun] Support Quant8 for Concat (#4352)

This commit supports `quant8` data type for `Concat` on `acl_cl` backend and remove skip related test list.

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

[neurun] Support other types for ConstantInitializer (#4349)

Some tests such as `GeneratedTest.conv_quant8` have failed cause of not supporting other types for `ConstantInitializer`.

This commit supports other types for `ConstantInitializer` and remove skip list `conv_quant8xxx`.

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

Support multiple indices upto 4D output ranks in Gather (#4339)

This commit supports below test cases:
2D input + 2D indices => 3D output
2D input + 3D indices => 4D output
3D input + 2D indices => 4D output

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Use shared pointer for operands and opertions in Plan (#4350)

Use shared pointer for operands and opertions in Plan
Define new contructor to set operands and operations
Remove setCompile method and friend class

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

Remove plan getter in Compiler (#4347)

Remove plan getter in Compiler
Prepare remove non-const model getter in Plan
Fix plain initialize in Compiler to return nullptr when release() is called before compile()

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

update `.gitignore` (#4337)

- update `.gitignore` to support `gnu-global` tool

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

[PACL] Support ABS_EX op (#4334)

This commit supports ABS_EX operation for PACL.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Support explicit padding and activation for Conv2D (#4330)

we can pass all of generated `Conv2D` tests(except `quant8` tests).

This commit supports explicit padding and activation for `Conv2D` to pass related tests.

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

Remove NOT_COMPILED state in compiler (#4332)

* Remove NOT_COMPILED state in compiler

Remove NOT_COMPILED in compiler's state
Make isCompiled flag to check compile result

* Rename to set compiled

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

Replace CLNormalizationLayerEX with CLNormalizationLayer (#4324)

This commit replaces CLNormalizationLayerEx of ARMComputeEx with CLNormalizationLayer of ARMCompute.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

fix bug (#4331)

- fix bug in case statement.
- change confusing(or maybe buggy) code to clear one.

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

Support Gather in all input ranks (#4312)

This commit supports Gather operation in all input ranks.
But it only supports 1D rank in indices.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

[neurun] Remove unnecessary header of operand (#4308)

Remove unnecessary header of `operand::Object` and added header
inclusion from the files that are depend on `operand::LowerInfo`.

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

Replace CLStridedSliceEx with CLStridedSlice (#4315)

* Replace CLStridedSliceEx with CLStridedSlice

This commit replaces CLStridedSliceEx of ARMComputeEx with CLStridedSlice of ARMCompute.

* Fix wrong initialization of std::vector

This commit fixes initialization of std::vector when compiling StridedSlice operation.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Fix `TensorInfo::total_size` (#4321)

Fix calculation of total size for a tensor to use actual element size.
Before it assumed the element size of 4.

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

Move compute from frontend to internal (#4310)

Move compute phase from frontend to Executor class
Remove not used methods in ANeuralNetworksExecution and Executor

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

[neurun] Fix to set wrong length for PermuteSource/Sink (#4300)

* [neurun] Fix setting wrong length for PermuteSource/Sink

Rleated : #2147

I've tried to run the generated tests `GeneratedTests.avg_pool_quant8_1~4`,since merged all related PRs.
But the tests are failed with wrong matching.

I figured out the reason of problem is from setting wrong length for `PermuteSource/Sink` by debugging.

This commit fixes setting wrong length for `PermuteSource/Sink` and removes skip lists for that tests.

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

[neurun] Change reshape kernel from `CLReshapeLayer` to `GenericReshapeLayer` (#4314)

Related : #2147

In #1353,

`GenericReshapeLayer` was introduced to remove height/width restriction, and `PACL` uses that now.
This commit changes reshape kernel from `CLReshapeLayer` to `GenericReshapeLayer`.

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

Avoid redefinition of macro functions (#4320)

Avoid redefinition of CAST_CL and CAST_NE that generates warning.

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

Fix concat for not rank 4 (#4317)

Now concat operation rank is not 4 can run using cpu implementation

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

[neurun] Move Model class to `model` (#4303)

Move Model class to `model` from `graph` since `Model` represents pure
neural network primitive elements.

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

Update the description of Gather (#4277)

This commit updates the description of the types that CLGather and CLGatherKerenl support.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Introduce BUILD_UBEN option (#4309)

* Introduce BUILD_UBEN option

This commit introduces BUILD_UBEN option to fix Tizen build break.

* Set default value

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

[PACL] Replace CLReduceOperation with CLReduceMean. (#4304)

This commit replaces CLReduceOperation with CLReduceMean in case of mean operation.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[PACL] Replace CLLogicalNot with CLBitwiseNot (#4299)

This commit replaces CLLogicalNot of ARMComputeEx with CLBitwiseNot of ARMCompute

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Move input and output setting into Executor class (#4293)

* Move input and output setting into Executor class

Move source (input setting) and sink (output setting) into Executor class

* Fix source list name to same as directory name

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

[neurun] Extract sizeOfDataType as a function (#4307)

Extract sizeOfDataType as a function which were implemented in class
`Object`.

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

Introduce uben (#4301)

* Introduce uben

This commit introduces uben (Micro BENchmark) suite based on nonius
benchmark framework.

Initial version includes convolution 3x3 (with unit stride) benchmark
over ARM Compute CL Direct/GEMM/Winograd kernels.

* Fix a bug in ker_info method

* Use CLWinogradConvolutionLayer correctly

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

[PACL] Replace CLPixelWiseDivision with CLArithmeticDivision (#4302)

This commit replaces CLPixelWiseDivision of ARMComputeEx with CLArithmeticDivision of ARMCompute.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Support activation for AvgPool2D (#4297)

Related : #2147

This commit supports unimplemented activation for AvgPool2D.

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

[PACL] Add the declaration of asTensorInfo (#4298)

This commit adds the declaration of asTensorInfo.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Support explicit padding for AvgPool2D (#4278)

* [neurun] Support explicit padding for AvgPool2D

Related : #2147

This commit supports explicit padding for `AvgPool2D`.

- Merge `Implicit` and `Explicit` `Node` compared to `PACL`
- Remove related test skip list

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* move explicit_padding to param, add TODO comment

Move source and sink in wrapper (#4290)

* Move source and sink in wrapper

Move source and sink setting and call in wrapper into Executor class

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Remove header include

[PACL] Replace CLBatchToSpaceND with CLBatchToSpaceLayer (#4283)

This commit replaces CLBatchToSpaceND of ARMComputeEx with CLBatchToSpaceLayer of ARMCompute

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Provide Q8 sup and comments correction for Arg operation (#4268)

1. Provide Q8 sup
2. comments correction

Signed-off-by: shubham <shub98.gupta@samsung.com>

[neurun] Support Relu1, Relu6 activation (#4287)

I've tried to pass `GeneratedTests.avg_pool_float_4`
However I met the missmatching in result.
I figured out it is the problem of `activation` Relu6.

This commit supports `Relu1` and `Relu6` activation in `ActivationBuilder`.

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

[neurun] Move kernels into backends (#4126)

* [neurun] Move kernels into backends

Kernels are part of backends, so they should be in subdir of their own
backends

* [neurun] Change namespaces of kernels to include backends

Exchange namespaces of backends and kernels in kernels

* [neurun] Add "backend" namespace to kernels

Add "backend" namespace to kernels

Signed-off-by: Poshshoev Dilshodzhon <d.poshshoev@samsung.com>

[PACL] Add an overloading function of asTensorInfo (#4284)

This commit addes an overloading function of asTensorInfo that can be set arm_compute::DataType optionally.

 Some kernels of ARMCompute require data type that does not exist in nnapi spec. However, the PACL currently
sets the data type of ARMCompute to a mapped data type of nnapi. There is a need for a function to optionally set
the data type of ARMCompute.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Move State member to Compiler from Plan (#4286)

Move State member variable to Compiler from Plan.
Plan is just part of compilation so it would be more proper.

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

Introduce executor class (#4281)

Introduce executor class to control execution phase on runtime internal

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

Update frontend execution API (#4282)

- Free execution object
- Return bad state when setInput & setOutput using memory object (NYI)

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

Remove Passing gtests from skiplist (#4269)

After integration of ACL v18.11, some gtests are
passing which were earlier failing.
This patch will remove all passing gtests from
skiplist.

Signed-off-by: shubham <shub98.gupta@samsung.com>

Changes in CLPermuteExKernel.cpp (#4273)

1. move calc shape func of validation in inside of
if condition since it is getting used only when that if
condition is true.
2. remove tensorshape local func since misc/shapecalc/utils has
same function which can be directly used.

Signed-off-by: shubham <shub98.gupta@samsung.com>

Update validation check on frontend execution (#4267)

* Update validation check on frontend execution

Update validation check on NNAPI frontend execution's input & output setting
- Check length
- Check shape
- Print error message

Abstract to access model
- Operand index of input & output
- Type & shape compare

* Fix bug

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

Update broken link in `howoto` document. (#4280)

- Updated link to Dockerfile which broken by directory restructuring.

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

Remove CLArithmeticSubtractionEx from ArmComputeEx (#4275)

This commit removes CLArithmeticSubtractionEx from ArmComputeEx.
  - remove CLArithmeticSubtractionEx
  - replace CLArithmeticSubtractionEx with CLArithmeticSubtraction since CLArithmeticSubtraction supports the broadcast.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Update broken links to model repository in `howto.md`. (#4274)

- Update broken model zoo links as `tflite` leaves contrib.

Signed-off-by: Sung-Jae Lee <sj925.lee@samsung.com>

[neurun] Remove skip list for TrivialTest.AddThree (#4266)

Related : #4211

This commit removes skip list for `TrivialTest.AddThree`, since #4263 and #4257 are merged.

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

[neurun] Support to initialize 2D constant (#4257)

* [neurun] Support to initialize 2D constant

Until now, `ConstantInitializer` in `neurun` has supported only for `FullyConnected`.
This commit supports to initialize 2D constant for all other operations.

* Change some comment

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

Move type converter (#4253)

* Move type converter

Move type converter from graph to util
It makes more independent internal code with NNAPI

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Fix test and remove deprecated include

* Fix comment

* Divide function declaration and definition

Use config manager for logging (#4261)

Use config manager for neurun logging enable/disable

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

Correction in comment in Pad kernel (#4262)

this patch will correct the comment which tells
what the pad kernel does.

Signed-off-by: shubham <shub98.gupta@samsung.com>

Renaming ArgMinMax to ArgOperation (#4238)

In order to have genericity so that any arg op
can be added, this patch will rename occurances
of ArgMinMax to ArgOperation.
Also, this patch will correct few of the comments.

Signed-off-by: shubham <shub98.gupta@samsung.com>

[neurun] Fix when `Graph::lower` is called twice (#4263)

This commit fixes to handle `Graph::lower` is called multiple times.

Fix #4258

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

[neurun] Remove skip list for TrivialTest.AddTwo (#4260)

Related : #4211

This commit removes skip lit for `TrivialTest.AddTwo`, since #4249 is merged.

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

Remove dependence of data format(NHWC) of PACL from kernels of PAD (#4014)

This commit removes dependece of data format(NHWC) of PACL runtime from kernels of PAD.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>