Skip to content
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.

Already on GitHub? Sign in to your account

[modified] modified app-installer process in plugin-parser and service-db #39

Merged

Conversation

yeonykim2
Copy link
Contributor

  • modified app-installer Handle process. - INSTALL, UNINSTALL, UPGRADE.
  • delete the installed models, including activated model, by rpk uninstall.

@yeonykim2 yeonykim2 force-pushed the dev/pkg-mgr-parser/modified-installer branch 2 times, most recently from eb84d5c to cb91e2b Compare April 3, 2024 02:17
@yeonykim2 yeonykim2 force-pushed the dev/pkg-mgr-parser/modified-installer branch from cb91e2b to 7260fa0 Compare April 4, 2024 03:52
daemon/service-db.cc Outdated Show resolved Hide resolved
In rpk uninstall, The target model will be deteted, even if the model is activated.
Add force-delete model in daemon.

Signed-off-by: Suyeon Kim <[email protected]>
@yeonykim2 yeonykim2 force-pushed the dev/pkg-mgr-parser/modified-installer branch 3 times, most recently from 3220053 to 144344b Compare April 5, 2024 08:10
- modified app-installer Handle process. - INSTALL, UNINSTALL, UPGRADE.
- delete the installed models, including activated model, by rpk uninstall.

Signed-off-by: Suyeon Kim <[email protected]>
@yeonykim2 yeonykim2 force-pushed the dev/pkg-mgr-parser/modified-installer branch from 03b6986 to c5b870f Compare April 5, 2024 08:21
Copy link
Member

@anyj0527 anyj0527 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM👍Thanks!

@jaeyun-jung jaeyun-jung merged commit 5b843df into nnstreamer:main Apr 5, 2024
9 of 10 checks passed
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.

3 participants