8000 Tizen 8 backport main by anyj0527 · Pull Request #6 · anyj0527/api · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Tizen 8 backport main #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

8000 Already on GitHub? Sign in to your account

Open
wants to merge 93 commits into
base: lts/1.8.4.b
Choose a base branch
from
Open

Tizen 8 backport main #6

wants to merge 93 commits into from

Conversation

anyj0527
Copy link
Owner

.,

jaeyun-jung and others added 30 commits September 19, 2023 09:11
Update version, start implementation for next release.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
- Add service_dep dependency to service related unittests.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
Remove armcl dependency when nnfw is enabled.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Code consistency, add tag in return val field.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Use 'caps' property in appsrc, not caps-filter.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
1. Update ml-remote function name, prefix 'ml_service'.
2. Hide internal enum and struct.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Add key to get the neural network name from ml-option.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Prevent error case if nns-edge dependency does not exists.
Add internal function to release ml-svc handle.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
This is a trivial fix to silence sign-compare warnings in a unit test
case for inference C-APIs

/usr/src/googletest/googletest/include/gtest/gtest.h:1545:11: error: comparison of integer expressions of different signedness: ‘const unsigned int’ and ‘const int’ [-Werror=sign-compare]
1545 |   if (lhs == rhs) {                                                                               |       ~~~~^~~~~~                                                                            cc1plus: all warnings being treated as errors

Signed-off-by: Wook Song <wook16.song@samsung.com>
This patch changes the name of ML-Agent from ml-agentd to ml-agent.

Signed-off-by: Wook Song <wook16.song@samsung.com>
There is some timing issue to save the changes in DB,
set trasnsation for INSERT/UPDATE command.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
1. Update 'activate' flag from json when registering new model info from res-pkg.
2. Allow null description in json, description string is not mandatory field.
3. Fix log message, hide app info string.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, fix typo and indent.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
We implemented 0-initialized dimension in nnstreamer.
Remove unnecessary rank list from single handle.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Add remote service event callback

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
We implemented 0-init dimension in nnstreamer.
If dim value at index of old rank limit (4) is not 0, it is extended.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Parse json file and update model/pipeline/resource data when installing res-pkg.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, fix indent and remove unnecessary space.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
We do not use total size of tensors.
Remove unnecessary variable for total data size.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
1. Create tensors-info handle before calling sink callback.
2. We do not need to use ml-info in pipeline elem struct. Use gst-info instead.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Use gst-info and create handle inside of single-shot instance.
Referencing struct does not initialize lock and other variables.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, use GstTensorMemory struct defined in nnstreamer.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
sync to native API, update max rank and tensor size limit.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, use public function to clone tensor-data handle.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Add a path to save the received model file to the ml-option.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Let's use G_DIR_SEPARATOR_S instead of "/".

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Let's generate edge handle ID automatically by default.
The user also set edge ID using ml_option_set().

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Use gst-info struct and util functions from nnstreamer.
Remove unnecessary functions for ml-info.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Check invalid dimension case before updating tensor-info handle.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Parse clear field from json, to delete old model or resource from database.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
niley7464 and others added 25 commits March 29, 2024 17:28
This patch allows 0 value dimension in setDimension function.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>
Remove unnecessary macro for old API set.
We should update Tizen test later.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
We can register new model (or pipeline) using ml-service API.
Add test for registered key from ml-agent.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, remove unnecessary code to create info handle for internal usage.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Fix tizen auto coverage measurement failure.
Use default root path instead of environment variable.
The model path of the config file is hard coded, so the wrong model path is passed to ml-service.

config_single_add.conf
```
{
    "single" :
    {
        "framework" : "tensorflow-lite",
        "model" : ["../tests/test_models/models/add.tflite"]
    }
}

```

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Add definition to get the install prefix to run the ml-agent for unittest.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Fix wrong logical operator to break the loop.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
 - Integrates remote service into ml-service
 - Separate the common part of ml-service-extension to ml-service and
ml-servie-extension.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Rename ml-service-remote to ml-service-offloading

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
Change ml-service node type from pub/sub to query server/client to send
reply.

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
 - Remove unnecessary error handling
 - Handle error case
 - Parse tensor data more than 1

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
This patch mainly revises the gradle and ndk-build build scripts
to support Android API Level 34 with GStreamer v1.24
(the latest GST release as of 2024.03).

Signed-off-by: Wook Song <wook16.song@samsung.com>
Code clean, fix svace issues and remove unnecessary code.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
This patch replaces TAOS-CI script for android build.

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>
This is a trivial patch that adds more negative test cases for the
inference C-APIs.

Signed-off-by: Wook Song <wook16.song@samsung.com>
nnstreamer-android-resource directory is changed not to download using scripts.
So it shouldn't be removed.

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

Signed-off-by: gichan2-jang <gichan2.jang@samsung.com>
To remove unnecessary dbus-sig, add force-delete flag in ml-agent interface.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, remove unnecessary code and update log before updating internal function (related to ml-offloading).

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
- Add function for training offloading
- Add function to transfer trained model
- Add node information to transmitted pipeline information
- set sink callback by node name of node information
- Add stop func for ready to complete model training
- Add test case

Signed-off-by: hyunil park <hyunil46.park@samsung.com>
Code clean, update function name for offloading and fix svace issues.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
Code clean, remove unnecessary code and fix svace issue.

Signed-off-by: Jaeyun Jung <jy1210.jung@samsung.com>
- Change url from Tizen to Tizen-8.0

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
@taos-ci
Copy link
Collaborator
taos-ci commented Apr 12, 2024

📝 TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #6. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

F249 @taos-ci
Copy link
Collaborator
taos-ci commented Apr 12, 2024

:octocat: cibot: @anyj0527, tests/plugin-parser/unittest_mlops_plugin_parser.cc does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

@taos-ci
Copy link
Collaborator
taos-ci commented Apr 12, 2024

:octocat: cibot: @anyj0527, The last line of a text file must have a newline character. Please append a new line at the end of the line in plugin-parser/pkgmgr-info.h.

@taos-ci
Copy link
Collaborator
taos-ci commented Apr 12, 2024

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

@taos-ci
Copy link
Collaborator
taos-ci commented Apr 12, 2024

:octocat: cibot: @anyj0527, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-api-anyj0527/ci/repo-workers/pr-checker/6-202404121455180.76249694824219-99c4ad8ea0f0df4493bb3f5abdcc8870add43902/.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants
0