* [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite @ 2020-03-06 17:46 Hongmei Gou 2020-03-06 18:26 ` Denys Dmytriyenko 0 siblings, 1 reply; 5+ messages in thread From: Hongmei Gou @ 2020-03-06 17:46 UTC (permalink / raw) To: meta-arago; +Cc: Hongmei Gou * This is for including the examples built from the tensforflow lite package to the target filesystem at usr/share/tensorflow-lite/examples * As tensorflow-lite is creating static library, tensorflow-lite-demo which depends on tensorflow-lite will not bring in the examples folder for tensorflow-lite to the target filesystem Signed-off-by: Hongmei Gou <a0271529@ti.com> --- This is applicable to thud only, since zeus/master is now using ti-analytics.bb .../packagegroups/packagegroup-arago-tisdk-addons.bb | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb index 583d2c0b..ff34ab95 100644 --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb @@ -1,6 +1,6 @@ DESCRIPTION = "Task to install additional utilities/demos for SDKs" LICENSE = "MIT" -PR = "r77" +PR = "r78" PACKAGE_ARCH = "${MACHINE_ARCH}" @@ -153,6 +153,7 @@ ARMNN_PACKAGES = "armnn" EXTRA_PACKAGES_append_ti33x = " voxelsdk \ arm-compute-library \ ${ARMNN_PACKAGES} \ + tensorflow-lite \ tensorflow-lite-demo \ pdm-anomaly-detection \ " @@ -160,6 +161,7 @@ EXTRA_PACKAGES_append_ti33x = " voxelsdk \ EXTRA_PACKAGES_append_ti43x = " voxelsdk \ arm-compute-library \ ${ARMNN_PACKAGES} \ + tensorflow-lite \ tensorflow-lite-demo \ pdm-anomaly-detection \ " @@ -169,12 +171,14 @@ EXTRA_PACKAGES_append_omap-a15 = " voxelsdk \ big-data-ipc-demo-linux-firmware \ arm-compute-library \ ${ARMNN_PACKAGES} \ + tensorflow-lite \ tensorflow-lite-demo \ pdm-anomaly-detection \ " EXTRA_PACKAGES_append_am65xx = " arm-compute-library \ ${ARMNN_PACKAGES} \ + tensorflow-lite \ tensorflow-lite-demo \ pdm-anomaly-detection \ " -- 2.17.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite 2020-03-06 17:46 [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite Hongmei Gou @ 2020-03-06 18:26 ` Denys Dmytriyenko 2020-03-06 22:26 ` Gou, Hongmei 0 siblings, 1 reply; 5+ messages in thread From: Denys Dmytriyenko @ 2020-03-06 18:26 UTC (permalink / raw) To: Hongmei Gou; +Cc: meta-arago On Fri, Mar 06, 2020 at 12:46:25PM -0500, Hongmei Gou wrote: > * This is for including the examples built from the tensforflow lite package > to the target filesystem at usr/share/tensorflow-lite/examples > > * As tensorflow-lite is creating static library, tensorflow-lite-demo which > depends on tensorflow-lite will not bring in the examples folder for > tensorflow-lite to the target filesystem Not really clear from this description - do you package examples in a separate package? Then you need to list that package in RDEPENDS packagegroup below. Otherwise the standard depency from demo package should be fine. Denys > Signed-off-by: Hongmei Gou <a0271529@ti.com> > --- > This is applicable to thud only, since zeus/master is now using ti-analytics.bb > > .../packagegroups/packagegroup-arago-tisdk-addons.bb | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb > index 583d2c0b..ff34ab95 100644 > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb > @@ -1,6 +1,6 @@ > DESCRIPTION = "Task to install additional utilities/demos for SDKs" > LICENSE = "MIT" > -PR = "r77" > +PR = "r78" > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > @@ -153,6 +153,7 @@ ARMNN_PACKAGES = "armnn" > EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > arm-compute-library \ > ${ARMNN_PACKAGES} \ > + tensorflow-lite \ > tensorflow-lite-demo \ > pdm-anomaly-detection \ > " > @@ -160,6 +161,7 @@ EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > EXTRA_PACKAGES_append_ti43x = " voxelsdk \ > arm-compute-library \ > ${ARMNN_PACKAGES} \ > + tensorflow-lite \ > tensorflow-lite-demo \ > pdm-anomaly-detection \ > " > @@ -169,12 +171,14 @@ EXTRA_PACKAGES_append_omap-a15 = " voxelsdk \ > big-data-ipc-demo-linux-firmware \ > arm-compute-library \ > ${ARMNN_PACKAGES} \ > + tensorflow-lite \ > tensorflow-lite-demo \ > pdm-anomaly-detection \ > " > > EXTRA_PACKAGES_append_am65xx = " arm-compute-library \ > ${ARMNN_PACKAGES} \ > + tensorflow-lite \ > tensorflow-lite-demo \ > pdm-anomaly-detection \ > " > -- > 2.17.1 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite 2020-03-06 18:26 ` Denys Dmytriyenko @ 2020-03-06 22:26 ` Gou, Hongmei 2020-03-10 15:34 ` Denys Dmytriyenko 0 siblings, 1 reply; 5+ messages in thread From: Gou, Hongmei @ 2020-03-06 22:26 UTC (permalink / raw) To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org > -----Original Message----- > From: Dmytriyenko, Denys > Sent: Friday, March 6, 2020 1:27 PM > To: Gou, Hongmei > Cc: meta-arago@arago-project.org > Subject: Re: [meta-arago] [thud] [PATCH] packagegroup-arago-tisdk-addons: > add tensorflow-lite > > On Fri, Mar 06, 2020 at 12:46:25PM -0500, Hongmei Gou wrote: > > * This is for including the examples built from the tensforflow lite package > > to the target filesystem at usr/share/tensorflow-lite/examples > > > > * As tensorflow-lite is creating static library, tensorflow-lite-demo which > > depends on tensorflow-lite will not bring in the examples folder for > > tensorflow-lite to the target filesystem > > Not really clear from this description - do you package examples in a separate > package? Then you need to list that package in RDEPENDS packagegroup > below. > Otherwise the standard depency from demo package should be fine. There used to be a separate tensorflow-lite-examples package for the examples. The examples got in the tensorflow-lite package after changing the target location to /usr/share/tensorflow-lite. So, the current tensorflow-lite package itself contains the examples (console output). On the other hand, we newly introduced the tensorflow-lite-demo package which contains demos with GUI display, and it depends on tensorflow-lite. > > Denys > > > > Signed-off-by: Hongmei Gou <a0271529@ti.com> > > --- > > This is applicable to thud only, since zeus/master is now using ti- > analytics.bb > > > > .../packagegroups/packagegroup-arago-tisdk-addons.bb | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup- > arago-tisdk-addons.bb b/meta-arago-distro/recipes- > core/packagegroups/packagegroup-arago-tisdk-addons.bb > > index 583d2c0b..ff34ab95 100644 > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago- > tisdk-addons.bb > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup- > arago-tisdk-addons.bb > > @@ -1,6 +1,6 @@ > > DESCRIPTION = "Task to install additional utilities/demos for SDKs" > > LICENSE = "MIT" > > -PR = "r77" > > +PR = "r78" > > > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > > > @@ -153,6 +153,7 @@ ARMNN_PACKAGES = "armnn" > > EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > arm-compute-library \ > > ${ARMNN_PACKAGES} \ > > + tensorflow-lite \ > > tensorflow-lite-demo \ > > pdm-anomaly-detection \ > > " > > @@ -160,6 +161,7 @@ EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > EXTRA_PACKAGES_append_ti43x = " voxelsdk \ > > arm-compute-library \ > > ${ARMNN_PACKAGES} \ > > + tensorflow-lite \ > > tensorflow-lite-demo \ > > pdm-anomaly-detection \ > > " > > @@ -169,12 +171,14 @@ EXTRA_PACKAGES_append_omap-a15 = " > voxelsdk \ > > big-data-ipc-demo-linux-firmware \ > > arm-compute-library \ > > ${ARMNN_PACKAGES} \ > > + tensorflow-lite \ > > tensorflow-lite-demo \ > > pdm-anomaly-detection \ > > " > > > > EXTRA_PACKAGES_append_am65xx = " arm-compute-library \ > > ${ARMNN_PACKAGES} \ > > + tensorflow-lite \ > > tensorflow-lite-demo \ > > pdm-anomaly-detection \ > > " > > -- > > 2.17.1 > > > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite 2020-03-06 22:26 ` Gou, Hongmei @ 2020-03-10 15:34 ` Denys Dmytriyenko 2020-03-10 18:29 ` Gou, Hongmei 0 siblings, 1 reply; 5+ messages in thread From: Denys Dmytriyenko @ 2020-03-10 15:34 UTC (permalink / raw) To: Gou, Hongmei; +Cc: meta-arago@arago-project.org On Fri, Mar 06, 2020 at 05:26:57PM -0500, Gou, Hongmei wrote: > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Friday, March 6, 2020 1:27 PM > > To: Gou, Hongmei > > Cc: meta-arago@arago-project.org > > Subject: Re: [meta-arago] [thud] [PATCH] packagegroup-arago-tisdk-addons: > > add tensorflow-lite > > > > On Fri, Mar 06, 2020 at 12:46:25PM -0500, Hongmei Gou wrote: > > > * This is for including the examples built from the tensforflow lite package > > > to the target filesystem at usr/share/tensorflow-lite/examples > > > > > > * As tensorflow-lite is creating static library, tensorflow-lite-demo which > > > depends on tensorflow-lite will not bring in the examples folder for > > > tensorflow-lite to the target filesystem > > > > Not really clear from this description - do you package examples in a separate > > package? Then you need to list that package in RDEPENDS packagegroup > > below. > > Otherwise the standard depency from demo package should be fine. > > There used to be a separate tensorflow-lite-examples package for the examples. > The examples got in the tensorflow-lite package after changing the target location > to /usr/share/tensorflow-lite. So, the current tensorflow-lite package itself contains > the examples (console output). > > On the other hand, we newly introduced the tensorflow-lite-demo package which > contains demos with GUI display, and it depends on tensorflow-lite. Sorry for the delay. So, tensorflow-lite-demo links *statically* against tensorflow-lite and hence there's no RDEPENDS preserved, which was expected to be there for the proper packaging to work... Is static linking the expected behavior? Are there plans to link dynamically? Second question - should examples from tensorflow-lite still be packaged separately? I understand they got moved and now picked up by the main package automatically. But there is an easy way to get the old behavior by prepending examples package to the PACKAGES list, instead of appending. I guess I can merge this to thud as is, but would like to clean this up going forward, so please answer above questions anyway. Thanks! Denys > > > Signed-off-by: Hongmei Gou <a0271529@ti.com> > > > --- > > > This is applicable to thud only, since zeus/master is now using ti- > > analytics.bb > > > > > > .../packagegroups/packagegroup-arago-tisdk-addons.bb | 6 +++++- > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup- > > arago-tisdk-addons.bb b/meta-arago-distro/recipes- > > core/packagegroups/packagegroup-arago-tisdk-addons.bb > > > index 583d2c0b..ff34ab95 100644 > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago- > > tisdk-addons.bb > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup- > > arago-tisdk-addons.bb > > > @@ -1,6 +1,6 @@ > > > DESCRIPTION = "Task to install additional utilities/demos for SDKs" > > > LICENSE = "MIT" > > > -PR = "r77" > > > +PR = "r78" > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > > > > > @@ -153,6 +153,7 @@ ARMNN_PACKAGES = "armnn" > > > EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > > arm-compute-library \ > > > ${ARMNN_PACKAGES} \ > > > + tensorflow-lite \ > > > tensorflow-lite-demo \ > > > pdm-anomaly-detection \ > > > " > > > @@ -160,6 +161,7 @@ EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > > EXTRA_PACKAGES_append_ti43x = " voxelsdk \ > > > arm-compute-library \ > > > ${ARMNN_PACKAGES} \ > > > + tensorflow-lite \ > > > tensorflow-lite-demo \ > > > pdm-anomaly-detection \ > > > " > > > @@ -169,12 +171,14 @@ EXTRA_PACKAGES_append_omap-a15 = " > > voxelsdk \ > > > big-data-ipc-demo-linux-firmware \ > > > arm-compute-library \ > > > ${ARMNN_PACKAGES} \ > > > + tensorflow-lite \ > > > tensorflow-lite-demo \ > > > pdm-anomaly-detection \ > > > " > > > > > > EXTRA_PACKAGES_append_am65xx = " arm-compute-library \ > > > ${ARMNN_PACKAGES} \ > > > + tensorflow-lite \ > > > tensorflow-lite-demo \ > > > pdm-anomaly-detection \ > > > " > > > -- > > > 2.17.1 > > > > > > _______________________________________________ > > > meta-arago mailing list > > > meta-arago@arago-project.org > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite 2020-03-10 15:34 ` Denys Dmytriyenko @ 2020-03-10 18:29 ` Gou, Hongmei 0 siblings, 0 replies; 5+ messages in thread From: Gou, Hongmei @ 2020-03-10 18:29 UTC (permalink / raw) To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org > -----Original Message----- > From: Dmytriyenko, Denys > Sent: Tuesday, March 10, 2020 11:35 AM > To: Gou, Hongmei > Cc: meta-arago@arago-project.org > Subject: Re: [meta-arago] [thud] [PATCH] packagegroup-arago-tisdk-addons: > add tensorflow-lite > > On Fri, Mar 06, 2020 at 05:26:57PM -0500, Gou, Hongmei wrote: > > > -----Original Message----- > > > From: Dmytriyenko, Denys > > > Sent: Friday, March 6, 2020 1:27 PM > > > To: Gou, Hongmei > > > Cc: meta-arago@arago-project.org > > > Subject: Re: [meta-arago] [thud] [PATCH] packagegroup-arago-tisdk- > addons: > > > add tensorflow-lite > > > > > > On Fri, Mar 06, 2020 at 12:46:25PM -0500, Hongmei Gou wrote: > > > > * This is for including the examples built from the tensforflow lite > package > > > > to the target filesystem at usr/share/tensorflow-lite/examples > > > > > > > > * As tensorflow-lite is creating static library, tensorflow-lite-demo > which > > > > depends on tensorflow-lite will not bring in the examples folder for > > > > tensorflow-lite to the target filesystem > > > > > > Not really clear from this description - do you package examples in a > separate > > > package? Then you need to list that package in RDEPENDS packagegroup > > > below. > > > Otherwise the standard depency from demo package should be fine. > > > > There used to be a separate tensorflow-lite-examples package for the > examples. > > The examples got in the tensorflow-lite package after changing the target > location > > to /usr/share/tensorflow-lite. So, the current tensorflow-lite package itself > contains > > the examples (console output). > > > > On the other hand, we newly introduced the tensorflow-lite-demo > package which > > contains demos with GUI display, and it depends on tensorflow-lite. > > Sorry for the delay. > > So, tensorflow-lite-demo links *statically* against tensorflow-lite and hence > there's no RDEPENDS preserved, which was expected to be there for the > proper > packaging to work... > > Is static linking the expected behavior? Are there plans to link dynamically? Yes, the static linking is expected - the upstream tensorflow lite package is creating static library. No plan to use dynamical linking for this quarter's release. > > > Second question - should examples from tensorflow-lite still be packaged > separately? I understand they got moved and now picked up by the main > package > automatically. But there is an easy way to get the old behavior by prepending > examples package to the PACKAGES list, instead of appending. Agree with you that moving the examples back as a separate tensorflow-lite-examples package is a better solution. > > I guess I can merge this to thud as is, but would like to clean this up going > forward, so please answer above questions anyway. Thanks! Thanks for merging this to thud! > > Denys > > > > > > Signed-off-by: Hongmei Gou <a0271529@ti.com> > > > > --- > > > > This is applicable to thud only, since zeus/master is now using ti- > > > analytics.bb > > > > > > > > .../packagegroups/packagegroup-arago-tisdk-addons.bb | 6 +++++- > > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/meta-arago-distro/recipes- > core/packagegroups/packagegroup- > > > arago-tisdk-addons.bb b/meta-arago-distro/recipes- > > > core/packagegroups/packagegroup-arago-tisdk-addons.bb > > > > index 583d2c0b..ff34ab95 100644 > > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup- > arago- > > > tisdk-addons.bb > > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup- > > > arago-tisdk-addons.bb > > > > @@ -1,6 +1,6 @@ > > > > DESCRIPTION = "Task to install additional utilities/demos for SDKs" > > > > LICENSE = "MIT" > > > > -PR = "r77" > > > > +PR = "r78" > > > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > > > > > > > @@ -153,6 +153,7 @@ ARMNN_PACKAGES = "armnn" > > > > EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > > > arm-compute-library \ > > > > ${ARMNN_PACKAGES} \ > > > > + tensorflow-lite \ > > > > tensorflow-lite-demo \ > > > > pdm-anomaly-detection \ > > > > " > > > > @@ -160,6 +161,7 @@ EXTRA_PACKAGES_append_ti33x = " voxelsdk \ > > > > EXTRA_PACKAGES_append_ti43x = " voxelsdk \ > > > > arm-compute-library \ > > > > ${ARMNN_PACKAGES} \ > > > > + tensorflow-lite \ > > > > tensorflow-lite-demo \ > > > > pdm-anomaly-detection \ > > > > " > > > > @@ -169,12 +171,14 @@ EXTRA_PACKAGES_append_omap-a15 = " > > > voxelsdk \ > > > > big-data-ipc-demo-linux-firmware \ > > > > arm-compute-library \ > > > > ${ARMNN_PACKAGES} \ > > > > + tensorflow-lite \ > > > > tensorflow-lite-demo \ > > > > pdm-anomaly-detection \ > > > > " > > > > > > > > EXTRA_PACKAGES_append_am65xx = " arm-compute-library \ > > > > ${ARMNN_PACKAGES} \ > > > > + tensorflow-lite \ > > > > tensorflow-lite-demo \ > > > > pdm-anomaly-detection \ > > > > " > > > > -- > > > > 2.17.1 > > > > > > > > _______________________________________________ > > > > meta-arago mailing list > > > > meta-arago@arago-project.org > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-03-10 18:31 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-03-06 17:46 [thud] [PATCH] packagegroup-arago-tisdk-addons: add tensorflow-lite Hongmei Gou 2020-03-06 18:26 ` Denys Dmytriyenko 2020-03-06 22:26 ` Gou, Hongmei 2020-03-10 15:34 ` Denys Dmytriyenko 2020-03-10 18:29 ` Gou, Hongmei
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.