[Build/Service] fix indent

Fix build error, clang-format of service-db.

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

[Service/Log] printing internal value

The ID from pipeline hashtable in daemon is internal value.
This PR hides this internal ID value.

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

[C-Api/CodeClean] remove unnecessary code

Code clean,
1. fix indent and remove unnecessary lines.
2. replace function to compare pkg type (case-insensitive).

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

[clang-format] line-comment length

clang-format, ignore line-comment length.

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

[Service] handle invalid version

Raise exception when given version number is 0.

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

Formatting/CC: Apply .clang-format to the C++ source files

This patch applies .clang-format to all of the .cc files, which are
source files written in C++.

find . -name "*.cc" -exec clang-format-15 --style=file:.clang-format {} \;

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

[C-Api] Tizen API guide

Update API description, fix design guide.

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

API/C: Add the missing header to the pipeline-internal header file

This patch adds a missing internal header, 'nnstreamer.h', which defines
data structures and types required by the
'ml-api-inference-pipeline-internal.h' file.

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

Tests: Add test cases for gdbus-util

This patch adds unit test cases for the helper functions in the
gdbus-util.c file.

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

[test] Add a negative testcase for ml_service_launch_pipeline

- Add an erroneous pipeline launch case with invalid tensor_filter

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Service] invalid table schema

Fix invalid table schema and cleanup service-db test.

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

[C-Api/CodeClean] remove duplicated code

Code clean, remove duplicated code.

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

[CodeClean] Change variable name

- Change variable name not to hide the parameter of cb

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[CodeClean] Fix memleak of pkg_info

- Free pkg_info handle when the callback is returned

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[TEST] Set `is-updatable` prop of single test

Set `is-updatable` prop of single test
 *line coverage: 80.1%

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

[build] Add missing errno.h in ml-agent-dbus-interface.c

- Add missing header <errno.h> to the source file.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

Meson/LCOV: Apply a workaround to increase lcov of 'service-db.cc'

This patch applies a simple workaround to increase the line coverage of
the 'service-db.cc' file.

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

MLAgent: Introduce libml-agentd

This patch defines APIs of the ML Agent Daemon whose backend is GDBus
and exports them to the outer space of the daemon so that the tightly
coupled dependency between the libcapi-ml-service and daemon is
eliminated.

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

[Test] remove unnecessary test

Code clean, remove empty testcase.

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

[Test/Meson] common dependency

Use predefined dependencies in meson script.

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

[Meson] dependency to tizen pkgs

Code clean, define dependencies to tizen packages.

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

[Build] error ml-svc on ubuntu

Fix error when building ml-svc on ubuntu.

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

Tizen] Exclude test files for SAM

Exclude test files for SAM

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

[C-API] revert fill dimension 1

This patch reverts fill dimension to default value 1.
In C# API, interop function ml_tensors_info_get_tensor_dimension
is used with length 4 dimension array instead of 16.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

Daemon/PkgMrg: Eliminate duplicate code blocks

This is a trivial patch for the elimination of the duplicate code
blocks.

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

Tests/PkgMgr: Add functions for emulation of Tizen's Pkg Manager APIs

This patch adds helper functions for the emulation of Tizen's Package
Manager APIs and utilities for unit test cases of ML Agent Service APIs.

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

[CodeClean] Fix memleak in pkgmgr

- Use g_autofree gchar to prevent memleak;

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[build] Add empty build dep for tizen appfw for local build/test

- Add an empty build dep for tizen appfw, used by servce API

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[pkgmgr] Parse rpk's "model_description.json" to fill app_info and register

- When rpk is installed, parse the provided model_description.json file
  and register models described in the json.

TODO: Handle when the rpk package is removed or updated.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[service] Check and parse app_info of tizen application (the caller)

In a Tizen application context,
`ml_service_model_register` fill app_info column. RPK cannot call this
API, so the value of key `is_rpk` is always false.

`ml_service_model_get*` APIs should parse the app_info and update the
path in case of models from 'RPK'.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

Tests/Dbus: Move DBus-related files to the tests/dbus directory

This patch relocates the dbus interface for the unit tests to the tests
directory. According to the changes, the meson build files are alse
updated as well.

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

[service] Add "app_info" column to the model DB

- Add a new column "app_info" to the service DB's tblModel table.
- This app_info is a json string consists of tizen application
  information which owns model.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

Tests/Meson: Revise meson build scripts in tests

This patch revises the meson build scripts related to the test
sub-directory.

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

[C-API] Fix set tensor dimension order

This patch fixes the order of ml_tensors_info_set_tensor_dimension.
If tensors_info is not extended, user may regard dimension array's length to 4.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

[agent] Change tizen log functions of ml-agent

- Let ml-agent logs be more verbose in Tizen.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[C-Api] Revert fill dim (default 1) commit

Because of native TC failure, this patch reverts the fill dim (default
1) commit.

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

[C-Api] fill dim (default 1)

When calling get-dimension(), fill remained dim - default dim is 1.

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

[Test] add tests to increase ML_TENSOR_RANK_LIMIT

This patch adds some TCs for increased ML_TENSOR_RANK_LIMIT.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

[C-API] Increase ML_TENSOR_RANK_LIMIT

This patch increases ML_TENSOR_RANK_LIMIT from 4 to 16.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

[CodeClean] fix indent

Trivial, fix indent and init null to global val.

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

[Single] Add input and output ranks

This patch adds input and output ranks to ml_single.
Input and output ranks are used when user explicitly sets rank property.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

[Service] init handle id

Init-zero to connect the signals of model-dbus.

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

[Service] common log util

Use common log util in model-dbus impl.

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

Meson/LCOV: Fix wrong code coverage ratio for ml-agent sources

The changes in 2b1df86 make two sets of object files for the
'machine-learning-agent' and 'machine-learning-agent-test' executables,
which leads to the wrong code coverage ratio for the ml-agent daemon
source files. This patch fixes it.

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

[CodeClean] Remove unnecessary code in service api test

- Remove some unused code.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent] Remove unused exceptions

- Let std::exception catch those runtime errors.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[CodeClean] Remove pipeline module related code from gdbus-util

- Move pipeline related code in gdbus-util to proper src.
- Adjust function names to the model-impl.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[test] Add some negative test for ml-agent

- Add some negative test for ml-agent.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

Meson: Add a missing build dependency, 'nnstreamer-internal'

This patch fixes build errors because of a missing header file,
'nnstreamer_internal.h', which should be included in the build
dependencies by meson.

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

[test] Add unittest for service DB

- Add negative testcases for service DB.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent/DB] Prevent sql injection by using `sqlite3_bind_text`

- Use `sqlite3_bind_text` instead of formatted string to prevent sql
  injection attack.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Build/Test] daemon for unittest

Cleanup build script, move test-daemon into 'tests' dir and fix file names.

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

[Service] do not free handle in error case

If called service-destroy function, do not free allocated handle if an error from dbus or database is raised.

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

[Build/Android] temporal block flatbuffer

Fix error, temporal block flatbuffer when building android library.

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

[test] Add negative tests for service APIs

- Add negative TC with DBUS unconnected.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[service] Change error code `not_support` into `io_error`

- If failed to connect with dbus, return ML_ERROR_IO_ERROR
  rather than ML_ERROR_NOT_SUPPORTED, which should be check with tizen
  feature.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[test] Fix mem leak of ml_option

- Add missing `ml_option_destroy`

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent] Print exception message

- Print exception message rather than simple error log

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[CodeClean] handle error case

Code clean, handle res-leak if error occurs.

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

[test][service] Add testcases using ml_service_model APIs

- Add some TCs using the new APIs.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[service/model] Implement new APIS ml_service_model_*

- Add build dependency for the json-glib in .spec and meson.build
- Implement new APIs using dbus glib apis.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[common/option] Implement new API ml_option_get.

- Implement ml_option_get which returns the paired value of given key
  in the ml_option instance.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[daemon] Implement dbus method callbacks

- Implement callback for each dbus methods:
  register, update_description, activate,
  get, get_activated, get_all, delete

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[daemon][DB] Update/Add service-db methods in sqlite3 backend

- Update set_model, get_model, delete_model methods.
- Add udpate_model_description, activate_model methods.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[daemon] Add dbus methods to handle model service APIs

- Update model-dbus.xml.
- Update the header interface dbus-interface.h

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Tizen] Fix gcno file path of gcov pacakge.

Tizen line coverage automation tool includes build dir path.
Fix gcno file path of gcov pacakge for compatibility with the tool.

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

[Service/Model] define new API

Define new function and info handle for machine learning models.

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

[Service] handle table version

Handle table version. If table schema is changed, we should implement data migration later.

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

[Service] validate model path

Validate model path when registering a model to database.
It should be an absolute path and not symbol link file.

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

[Test] name to root directory

Fix name to root directory, use prefix 'MLAPI_'.

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

[Build] remove dependency to leveldb

Change database to sqlite, remove dependency to leveldb.

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

[service] Add TCs using ml_service_delete_pipeline

- Add some negative testcases.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent][DB] Fix "delete_pipeline" to check the number of changed rows

- Check the number of changed rows after the DELETE query

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Service/CodeClean] separate method to init database

Code clean, sepatate method to initialize ml-service database.

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

[Build] dependency to pkgmgr on ubuntu

Fix build error on ubuntu, dependency to pkgmgr.

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

[Service/PkgMgr] Add skeleton code for the package manager

To get the event from the package manager, the callback function should
be registered in ML Agent. This patch adds the skeleton code to get the
install, uninstall, and upgrade event of the resource package from the
package manager. Related work such as update the database of ML Agent
should be needed.

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

[service] Add simple test using the new API

- Add a simple TC using API `ml_service_model_register`

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[service] Initial commit for the C-API `ml_service_model_register`

- Initial commit for the new API, which register model

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[service] Add util function to get model daemon proxy

- Add function to get proxy of the model d-bus interface
- "mlsm" stands for MachineLearningServiceModel

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent] Add initial commit for "model_register" method to the daemon

- Add initial commit of "model_register" method.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[agent] Rename a util function getting dbus proxy

- Rename `_get_proxy_new_for_bus_sync` -> `_get_mlsp_proxy_new_for_bus_sync`
- `mlsp` stands for MachineLearningServicePipeline

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[dbus] Add DBus interface for model management

- Add Register, Update, Rollback, SetInference, Get, GetLatest interfaces.
  These will be used by new ml-service APIS for model management.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[daemon] Remove outdated dbus methods

- Remove outdated dbus call for model (get_path, set_path, delete)
- Remove related testcases.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Service] replace ml-service DB

Replace ml-service database to sqlite3.

TODO:
- handle database version.
- implement methods for model info.

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

[Single] Fix coverity issue

This patch adds the error number for the missing argument of the format
specifier.

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

c-api: Update API error messages

This fixes https://github.com/nnstreamer/nnstreamer/issues/4015

that didn't give appropriate error messages.

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

[Service/CodeClean] change class name

Code clean, we will change the database later - level-db to sqlite.
Change class name and remove interface to ML-Service DB.

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

[Service/CodeClean] handle error case

Code clean, handle error case in service-db.

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

[Tizen/PPM] deprecated API from 7.5

From Tizen 7.5, privacy-privilege-mgr API will be deprecated.

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

[Service] dependency to sqlite

Add dependency to sqlite and cleanup, mark todo to remove leveldb for future work.

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

[C-Api/Single] invalid data handle in invoke thread

Prevent error case - null data ptr in invoke thread.

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

[Single] prevent use-after-free case

Clone input data in single-shot, to prevent use-after-free case when invoking a model with timed out option.

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

[C-Api] new function to copy data handle

Add new util function to copy tensor data.

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

[Service] handle dbus error case

Check returned val and handle error case when failed to invoke dbus methods.

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

[C-Api] fix typo

Fix typo in sample code for custom easy filter.

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

riscv64: Add RISC-V support

Add riscv64 architecture to the supported arch list.

Signed-off-by: Marek Pikuła <m.pikula@partner.samsung.com>

[Single] set tensors-info in output handle

The output tensor does not contain the tensors-info when invoking the model.
Update in/out tensors-info in single-shot handle.

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

[Test] feature state in tizen test

Set feature state after all testcases are done.

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

[gcov] Exclude test gcno files

Exclude test related gcno files from gcov package.

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

[Build] error on old ubuntu

Fix typo and build error on olb ubuntu.

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