[Android/Doc] update build guide

Update build guide with prebuilt gstreamer-1.16.1 binaries.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single] Added API and skeleton for flexible input dimension

Adding support for flexible input dimension
Added API and skeleton for setting input dimension via single API

V2:
Added more description for retval of ML_ERROR_NOT_SUPPORTED in documentation

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Decoder/Meson] Meson build for image segmentation decoder

In order to add tensor decoder for image segmentation to build tree,
updates meson build script.

Signed-off-by: niklasjang <niklasjang@gmail.com>
Signed-off-by: Jihoon Lee <ulla4571@gmail.com>

[WIP] Tensor decoder for image segmentation

In order to implement tensor decoder for image segmentation,
this patch composes skeleton code for it.

Signed-off-by: niklasjang <niklasjang@gmail.com>
Signed-off-by: Jihoon Lee <ulla4571@gmail.com>

[CodeClean] replace meaningless variables
rename a and b to other more meaningful ones.
a -> from, b -> to

Signed-off-by: Dongwon Lee <dongwon1103@naver.com>

[Typo] Fix typo

This patch fixes typo in packaging/nnstreamer.spec

Signed-off-by: Injae Kang <abcinje@gmail.com>

[CodeClean] combine if statements
combine if statements in tensor_transform.c.
I confirmed it worked well in example_image_classification_tflite.

Signed-off-by: Junsang Mo <junsang.mo@samsung.com>

Add execution bit of gradlew
Added excution bit of gradlew to prevent permission denied error.

Signed-off-by: 이원준 <dldnjs1013@nate.com>

[Security] handle returned value

Check mem allocation failure case and refactor file-close in IIO.

This PR is related with ssms-1016-0006.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[tests] Install models/data with tests

Install models/data with tests correctly when needed
Install tizen-capi-unittest with install-test option

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

Added the coverity badge

This commit is to append the badge of the Coverity (static analysis tool).
TODO: the Coverity module of TAOS-CI will be soon enabled in the nnstreamer GitHub repository.

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>

[Security] [Converter] handle text format in chain function

get min block size and use memcpy when input mime-type is text.

This PR is related with ssms-1016-0005.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Security] python lib name format

Define macro for file format and remove macro(preprocess defines) in the function call.

This PR is related with ssms-1016-0007.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Test] update model in tensor-filter

Add testcase to re-open nn model in tensor-filter using gst-harness.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[tests] Locate all models and data together

Locate all the models and data together for easier management

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[CodeClean]remove duplicates in TFLiteCore::getTensorDim
make a variable for interpreter->tensor (tensor_idx)->dims.
replace interpreter->tensor (tensor_idx)->dims with the variable.

Signed-off-by: Junsang Mo <junsang.mo@samsung.com>

[Doc] fix doxygen link

update doc-doxygen, fix the linked url.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Coverity/1087163] Fix dereference before null check

This intends to fix:
```
There may be a null pointer dereference, or else the comparison against null is unnecessary.

In ml_tensors_data_create: All paths that lead to this null pointer comparison already dereference the pointer earlier (CWE-476)
```

It appears that Coverity hates defensive coding.

This adds missing free on _data as well.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[tizen/nnfw] Add invoke for nnfw

Added invoke implementation for nnfw along with some bug fixes

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Security] returned value to compare str

Add one more commit for string compare, check returned value.

EXP20-C. Perform explicit tests to determine success, true and false, and equality

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android] add log to print version string

add log to print nnstreamer, gstreamer and glib version info.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Doc] fix typo and link

Update docs (fix links and typo in document)

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Typecast] Use explicit type casting with g_ascii_strto(u)ll() function

This patch use explicit type casting between variables and return values of
g_ascii_strtoll() or g_ascii_strtoull() functions.

This resolves issue #1762, which is about potential warnings
from compilers with strict option.

Signed-off-by: Injae Kang <abcinje@gmail.com>

[tizen/nnfw] Add get_input/output_dimensions

Added implementation for get_input/output_dimensions functions

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[CodeClean] remove redundant check for LLVM compatibility

As bdata->mode  is defined as bounding_box_modes type, it is never lower than zero.
Because find_key_strv()'s return value is definded as 'The count of replaced. (Set NULL if it is unnecessary)',
it does not affect.

And not latest clang complains about this redundance.(LLVM)

Signed-off-by: Niklas Jang <niklasjang@gmail.com>

[test]Change num_buffers type from guint to gint

It is dangerous to cast from gint to guint and vice versa.
According to Gstreamer GstBaseSrc menual, num-buffer's type is  'gint' not 'guint'.
num-buffers means 'Number of buffers to output before sending EOS (-1 = unlimited).
<https://gstreamer.freedesktop.org/documentation/base/gstbasesrc.html?gi-language=c#GstBaseSrc:num-buffers>

Signed-off-by: Niklas Jang <niklasjang@gmail.com>
[test]Change buffer_index type from guint to gint

There is comparison between buffer_index and num_buffers.
So buffer_index must be gint type like num_buffers.

Signed-off-by: Niklas Jang <niklasjang@gmail.com>

[Doc] Tizen Native App Guide

This guide allows developers to start writing Tizen native apps more easily.

This partially resovles #1751.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Security] cond to compare strings

EXP20-C. Perform explicit tests to determine success, true and false, and equality

This will fix SSMS-0003.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Security] check buffer size

check the output buffer size before copying the mem block.

This will fix SSMS-0002.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Api/Tizen] handle dpm restriction case

Check the pipeline with device policy restricted case.
Create dpm handle and when trying to start the pipeline, check device policy and handle error case.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Doc] Guide: writing a new tensor-filter subplugin

With https://github.com/nnsuite/nnstreamer-example/pull/68
this guide helps developer write a new filter subplugin easily.

With that PR in nnstreamer-example, this resovles #1746

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[CodeClean] Apply regular expressions

Use regular expressions instead of combined if-statements.

Signed-off-by: Injae Kang <abcinje@gmail.com>

[Filter/TF-lite] cleanup hw-type string

1. remove duplicated src and fix indent
2. move hw-type string definition

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[API/Tizen] Fix the rootstrap buildbreak issue

This patch fixes the rootstrap buildbreak issue by using relative path
instead of absolute one.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Api/Tizen] remove code which occurs error

remove code which occurs error.
Since the previous code lock the resource internally,
when another pipeline want to contact to the resource, an error is generated.

Signed-off-by: Hyoung Joo Ahn <hello.ahn@samsung.com>

[Api/Tizen] bug-fix to replace str

bug-fix to replace element string in pipeline description.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Api/Tizen] update restricted elements

Add tv-camsrc and pulsesrc as cam input src element.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Filter/Sub] use glib to compare strings

change string util functions to parse and compare strings.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Api/Tizen] Tizen internal API to construct pipeline

Add Tizen internal API without checking the permission.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single] Minor updates for modularity

Added minor updates in C-API single shot implementation for improved modularity

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Android/Test] add tc to check max local ref

Add aging test to check local ref table overflow.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Filter/Sub] already loaded case

Check returned value when the nn model file is already loaded.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Tizen/Api] fix invalid params to parse mm info

Add missed param mm-handle to get video cam src element in Tizen.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[API/Tizen] Remove version number from libcapi-nnstreamer.so

This patch removes the version number from libcapi-nnstreamer shared
object.
* libcapi-nnstreamer.so.0.3 -> libcapi-nnstreamer.so

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Tizen/nnfw] Add open/close callbacks for tensor_filter

This validates integration mechanisms for nnfw-runtime of Tizen.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[android/build] Update build script

Update build script to be directly used by CI for building

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Code Clean] Make example launch line in remark follow convention.
1. Move 'tensor_mux name=mux ! fakesink' from front to back.
2. Add \ for readability
3. Add missing regular expression ']|'

Signed-off-by: niklasjang <niklasajang@gmail.com>

[Filter/C++] Skeleton header

A few users have requested the ability to write C++ custom filters
especially for OpenCV usage.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Api/Tizen] internal function to ignore permission

add flag to ignore the permission (supposed tizen internal)

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single] Increase default timeout

Increase the default timeout to pass the test case on a slow machine/not fully
optimized tensorflow-lite. Also, as the running time for tf-lite test case in
aarch64 is over 10x compared to armv7l, higher defailt timeout is better.

In unit-test, increased the corresponding default timeout as well.
Further, when running many test cases in parallel, default timeout is scaled
with number of parallel runs.

Solves #1717

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Filter/nnfw] Tizen-nnfw skeleton

Skeleton code for Tizen-nnfw is added.
New nnfw of Tizen has its own interface; thus, we can no longer
rely on tensorflow-lite/nnapi for this.

Changes v2:
- Enabled nnfw by default for Tizen
- Applied jaeyun's comments (meson script style fix, nnfw subplugin explicitness of properties)

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Release of nnstreamer 1.0.0 (1.0 RC2)

This is 1.0 RC2.
With Tizen Official Release, 1.0 stable will be released.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Build] build fail in 18.04

missed typecast in single-shot, fix build error.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Coverage] remove meson private folder

remove meson-related directory in coverage report.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android/Test] tc fail case

Sometimes testcase failed with pipeline state NULL when stopping the pipeline.
Add sleep time before closing the pipeline.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Tests] Make tool scripts written in python be compatible with python3

There are several platforms which do not support python2.7 by default
anymore. To prepare this issue, this patch revises all python scripts in
tests to be also compatible with python3.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Android/Test] update testcases

1. update testcases for each java class
2. add flag to generate coverage report

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[API/Tizen] Update the comment for Tizen API

When pipeline string directly accesses the camera or recorder device of
the Tizen, appropriate privilege should be written on configuration
file. This patch new adds the 'remarks' comment for this privilege.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[svace/iio] security fixes related to sscanf

1. Fix security issue related with sscanf. Use g_ascii_str instead
Issue: WID:7938408 Use of vulnerable function 'sscanf' at tensor_src_iio.c:690. This function is unsafe.
Issue: WID:7938409 Use of vulnerable function 'sscanf' at tensor_src_iio.c:723. This function is unsafe.
2. Add comments for switch cases where breaks have been skipped
Issue: Missing break in the switch case

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Api/Tizen] check cam privilege

Check camera and recoder privilege for each src element.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Tests/TFLite] Run NNAPI tests only on Tizen

NNAPI tests are only needded in the case of Tizen. Although these tests
are not harmful on the other platform, this patch makes those NNAPI
tests run only on Tizen.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Tests/RepoDynamicity] Support running tests on Yocto

As a workaround for running tests on Yocto, this patch uses the default
installation directory excepting that the tests are running at the build
time or the directory where the test binary is located is provided.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Security] handle return value

Fix svace issues, check returned value and file descriptor.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android/Sample] move sample app to nnstreamere-example repo

Remove sample app in this repo.
I newly uploaded the sample code to use Android API in example repository.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android/Test] script to run unittest

Update script to run the unittest on target device.

Note that, the testcases are android instrumented test which requires the target device.

Cmd to build and run test: ./build-android-lib.sh --run_unittest=yes

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android/Api] add readme to build api

add readme to build android library using the script.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Tests/CustomFilters] Use default installation direcotory

Tests which require custom shared library does not consider the
tests using the unittest package (which means not the build time test).
They uses hardcoded path to indicate the location of the shared library.
In order to fix this problem, this patch makes the test use the default
installation directory when the test is running using the unittest
package.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[single-new] Memory leak bug fix in error cases

Added memory leak bug fix in error cases running

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Android/Api] update condition

1. update condition to check the given params.
2. update api description about ml model.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single-new] Remove dependency over pthread

Remove the dependency over pthread
Use glib functionalities instead

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Android/Api] fix typo

Fix typo in tensor-info class and update related sample code.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android/Test] base code to run instrumented test

Add base code to run Android instrumented test on the target devices.

TODO add testcases for each java class

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[ReadMe] Correct typo in bullet items under Getting Started

This patch corrects typo in bullet items under the Getting Started
section.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[CAPI/New-Single] Code Clean

For the readability and to prevent mistakes in the future,
- Apply symmetricity to lock/unlock, hide magic lock from programmers
- Apply common ops into macro

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Build] remove warning message

function declaration to avoid build warning (missing-declaration)

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Tests/Plugins] Avoid warnings caused by a GCC bug related to gtest

A GCC bug [1] might cause warnings related to 'conversion-null' when
using googletest macros with !FALSE as arguments. Therefore, in the case
that Werror is true, the build would be broken by this bug. To avoid
such situation, this patch replaces EXPECT_EQs in the plugin unit tests
with EXPECT_TRUE or EXPECT_FALSE.

[1] https://github.com/google/googletest/issues/322

Signed-off-by: Wook Song <wook16.song@samsung.com>

[single] Thread safety bug fix

Added bug fix for thread safe usage
Single shot API use single handle lock for making all its API call thread safe
However, the call to close the API destroys the handle itself making the
call to close not thread safe
Added a global lock to make closing of handles thread safe

Further, all the single handle locks are acquired inside the global lock
to ensure that single handle is not closed while the magic has been verified
and the local lock is not acquired

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Clean/SAM] add common functions

1. define common functions to reduce complexity
2. condition to print log message about nnapi

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[build] Added cleanup after android/ubuntu build

Added cleanup of patches after android/ubuntu build
Changed relative path of files to patches in patch file

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[build] Use patch file for modification in build

Use patch file to modify the nnstreamer.h when building for non-tizen systems

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Readme] Update deployment status

We now release macoS and JCenter binaries.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Build] cleanup script

Fix typo and cleanup build options.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Config] remove unnecessary lines

Remove unnecessary lines about tensorflow optimization.
The key of mem-optimize was removed and these lines are now unnecessary.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Single] Added unittests

Added more unittests for single-shot API testing
1. Multiple runs in parallel with varying timeouts
2. Changing timeout/closing the handle while calling the API in a different thread
3. Closing the handle multiple times
4. Check valid result after increasing the timeout, when earlier it was timing out

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Filter] add common functions

Add common functions for tensor-filter and filter-single.
1. add struct for common property
2. reduce duplicated code of tensor-filter

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Filter/Sub] use glib-mutex

remove pthread header and use g_mutex instead.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Api/Tizen] restricted elements in Tizen

Update the restricted elements in Tizen.

Restricted elements
- capsfilter input-selector output-selector queue tee valve
- appsink appsrc
- audioconvert audiorate audioresample audiomixer
- videoconvert videocrop videorate videoscale videoflip videomixer compositor
- fakesrc fakesink filesrc filesink
- audiotestsrc videotestsrc
- jpegparse jpegenc jpegdec pngenc pngdec
- tcpclientsink tcpclientsrc tcpserversink tcpserversrc udpsink udpsrc
- xvimagesink ximagesink evasimagesink evaspixmapsink glimagesink (mmfw ini)
- theoraenc lame vorbisenc wavenc volume oggmux avimux matroskamux (mmfw ini)
- v4l2src avsysvideosrc camerasrc fimcconvert (mmfw ini)

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[CodeClean] remove duplicates

remove duplicates and unnecessary code.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single] Static linking with unit-test

Updated meson to do static linking with unit-test executable
This resolves #1653

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Android/Api] add timeout in single-shot

Add method to set the timeout in single-shot API.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Api/Single] invalid condition to check file extension

1. Fix invalid cond to check file extension.
2. Add proper log message and error code to set tensors info.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[single-shot] Added support for timeout in single-shot

Added support for timeout in new single-shot API implementation to
comply with the existing single-shot API

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Api/Single] clear old buffer

1. Add mutex-lock for internal process.
2. Clear old buffer of appsink before pushing new buffer to appsrc.

When old buffer received in appsink after timeout, next output of invoke() may be the previous result.
To prevent this, clear old sample of appsink before pushing new buffer.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Android] update script to release

Add dependency and args for jcenter release.

To upload NNStreamer library, run the script with bintray user name.

./api/android/build-android-lib.sh --release=yes --release_version=$release_ver --bintray_user_name=$user_name --bintray_user_key=$user_key

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>

[Dist/RPM] Added library required by devel package

Added dependency of devel-capi on capi

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[single-shot] Added README.md

Added README.md with small details on the source files and
comparison on the single-shot APIs

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[single-shot] benchmarking the latency

Added benchmarking the latency and running times in the test code
This compares old vs new singleshot api - showing reduction in latency and running-time

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Doc] Update the component-description

This document was way too obsolete. Update it with recent info.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[single-shot] Completion for new single-shot API

Completed the implementation for single-shot API without GStreamer
Added a separate binary for this API
Added remaining functions in tensor_filter_single.c
Added another test run in meson and nnstreamer.spec for testing of this new implementation

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[TEST] Fix nnapi test

Fix nnapi test (#1647)

**Self evaluation:**
1. Build test:  [X]Passed [ ]Failed [ ]Skipped
2. Run test:  [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon <jijoong.moon@samsung.com>