On Tue, Nov 27, 2018 at 02:39:19PM -0500, Su, Qin wrote:
>
>
> >-----Original Message-----
> >From: Dmytriyenko, Denys
> >Sent: Monday, November 26, 2018 5:08 PM
> >To: Su, Qin
> >Cc: [email protected]
> >Subject: Re: [meta-arago] [PATCH v2] armnn: bump SRCREV and add a demo
> >example
> >
> >On Wed, Nov 21, 2018 at 04:56:35PM -0500, Qin Su wrote:
> >> updated to the latest version of 18.08 updated to generate versioned
> >> library created a demo example with mobileNet
> >>
> >> Signed-off-by: Qin Su <[email protected]>
> >> ---
> >
> >Would be nice to document here changes between patch revisions, i.e. v1 -> v2
>
> Will add a cover letter to explain the changes in patch v3
Cover letter is not required for a single patch. It is recommended for
multiple patches in a series to explain overall goal.
Patch revisions are documented briefly within the patch itself after the
commit log and after the "---" line, i.e. right here in context.
> >> ...0003-add-more-test-command-line-arguments.patch | 75 ++
> >> .../armnn/0004-generate-versioned-library.patch | 63 ++
> >> .../0005-add-armnn-mobilenet-test-example.patch | 69 ++
> >> .../armnn/0006-armnn-mobilenet-test-example.patch | 886
> >+++++++++++++++++++++
> >> .../recipes-support/armnn/armnn_git.bb | 40 +-
> >> 5 files changed, 1125 insertions(+), 8 deletions(-) create mode
> >> 100755
> >> meta-arago-extras/recipes-support/armnn/armnn/0003-add-more-test-
> >comma
> >> nd-line-arguments.patch create mode 100755
> >> meta-arago-extras/recipes-support/armnn/armnn/0004-generate-versioned-
> >> library.patch create mode 100755
> >> meta-arago-extras/recipes-support/armnn/armnn/0005-add-armnn-mobilenet
> >> -test-example.patch create mode 100755
> >> meta-arago-extras/recipes-support/armnn/armnn/0006-armnn-mobilenet-tes
> >> t-example.patch mode change 100644 => 100755
> >> meta-arago-extras/recipes-support/armnn/armnn_git.bb
> >
> >^^^^^^^^^^^^^^^^^^^^^^
> >Why are you making all patches and the recipe executable?
> >
> I will change them to 100644 in patch v3.
>
> >[snip]
> >
> >
> >> diff --git a/meta-arago-extras/recipes-support/armnn/armnn_git.bb
> >> b/meta-arago-extras/recipes-support/armnn/armnn_git.bb
> >> old mode 100644
> >> new mode 100755
> >> index 726a9da..2b25da8
> >> --- a/meta-arago-extras/recipes-support/armnn/armnn_git.bb
> >> +++ b/meta-arago-extras/recipes-support/armnn/armnn_git.bb
> >> @@ -1,23 +1,39 @@
> >> SUMMARY = "ARM Neural Network SDK"
> >> DESCRIPTION = "Linux software and tools to enable machine learning
> >(Caffe/Tensorflow) workloads on power-efficient devices"
> >> -LICENSE = "MIT"
> >> +LICENSE = "MIT & Apache-2.0"
> >> LIC_FILES_CHKSUM =
> >"file://LICENSE;md5=3e14a924c16f7d828b8335a59da64074"
> >> +LIC_FILES_CHKSUM_tfmodel =
> >"file://tfmodel/LICENSE;md5=7e2764bb89441ae65b52f305ffee5fa8"
> >
> >This is incorrect - why do you use _tfmodel override here? And it won't work
> >anyway. You simply need to add both entries to the variable.
> >
> Will fix this as the following in patch v3.
> LIC_FILES_CHKSUM = "file://LICENSE;md5=3e14a924c16f7d828b8335a59da64074 \
>
> file://../tfmodel/LICENSE;md5=7e2764bb89441ae65b52f305ffee5fa8"
Thanks.
> >BTW, tfmodel is a new entry in v2 - it wasn't there in v1...
> >
> Tfmodel (tensorflow model) was added to get the LICENSE file for Apache-2.0
> for mobilenet (which is one of the tensorflow models).
I guess this is an oversight of whomever making the mobilenet tarball releases
to not include license in the release. It is probably an overkill to pull a
new git repo just for the license file. You can refer the standard license as
file://${COMMON_LICENSE_DIR}/Apache-2.0 and add a comment that it applies to
mobilenet tarball.
> >> -PV = "20180502"
> >> +PV = "18.08"
> >> +PV_MAJOR = "${@d.getVar('PV',d,1).split('.')[0]}"
> >>
> >> BRANCH = "master"
> >> -SRCREV = "4c7098bfeab1ffe1cdc77f6c15548d3e73274746"
> >> +SRCREV = "c577f2c6a3b4ddb6ba87a882723c53a248afbeba"
> >> +SRCREV_tfmodel = "d299118e54aade794d5d98d9122b392783faeb8a"
> >> +SRCREV_tidl-api = "7e9a3942ec38efd64d45e34c10cba2f2938f5618"
> >> +
> >> +SRCREV_FORMAT = "armnn"
> >>
> >> S = "${WORKDIR}/git"
> >>
> >> inherit cmake
> >>
> >> SRC_URI = " \
> >> - git://github.com/ARM-software/armnn.git;branch=${BRANCH} \
> >> +
> >> + git://github.com/ARM-software/armnn.git;name=armnn;branch=${BRANCH}
> >> + \
> >> file://0001-stdlib-issue-work-around.patch \
> >> file://0002-enable-use-of-boost-shared-library.patch \
> >> + file://0003-add-more-test-command-line-arguments.patch \
> >> + file://0004-generate-versioned-library.patch \
> >> + file://0005-add-armnn-mobilenet-test-example.patch \
> >> + file://0006-armnn-mobilenet-test-example.patch \
> >> +
> >git://github.com/tensorflow/models;name=tfmodel;branch=${BRANCH};subdir=
> >${WORKDIR}/tfmodel;destsuffix=tfmodel \
> >> +
> >http://download.tensorflow.org/models/mobilenet_v1_2018_02_22/mobilenet
> >_v1_1.0_224.tgz;name=mobilenet;subdir=${WORKDIR}/tfmodel;destsuffix=tfmo
> >del \
> >> +
> >> + git://git.ti.com/tidl/tidl-api.git;name=tidl-api;branch=${BRANCH};su
> >> + bdir=${WORKDIR}/tidl-api;destsuffix=tidl-api \
> >> "
> >>
> >> +SRC_URI[mobilenet.md5sum] = "d5f69cef81ad8afb335d9727a17c462a"
> >> +SRC_URI[mobilenet.sha256sum] =
> >"1ccb74dbd9c5f7aea879120614e91617db9534bdfaa53dfea54b7c14162e126b"
> >> +
> >> DEPENDS = " \
> >> boost \
> >> protobuf \
> >> @@ -25,6 +41,7 @@ DEPENDS = " \
> >> arm-compute-library \
> >> armnn-caffe \
> >> armnn-tensorflow \
> >> + opencv \
> >> "
> >>
> >> RDEPENDS_${PN} = " arm-compute-library protobuf boost "
> >> @@ -39,22 +56,29 @@ EXTRA_OECMAKE=" \
> >> -DARMCOMPUTENEON=1 \
> >> -DBUILD_TESTS=1 -DPROFILING=1 \
> >> -DTHIRD_PARTY_INCLUDE_DIRS=${STAGING_DIR_HOST}${includedir} \
> >> + -DBUILD_ARMNN_EXAMPLES=1 \
> >> + -DGENERIC_LIB_VERSION=${PV} -
> >DGENERIC_LIB_SOVERSION=${PV_MAJOR} \
> >> "
> >>
> >> +TESTVECS_INSTALL_DIR = "${datadir}/arm/armnn"
> >> +
> >> do_install_append() {
> >> CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
> >> install -d ${D}${bindir}
> >> find ${WORKDIR}/build/tests -maxdepth 1 -type f -executable -exec cp
> >$CP_ARGS {} ${D}${bindir} \;
> >> cp $CP_ARGS ${WORKDIR}/build/UnitTests ${D}${bindir}
> >> + install -d ${D}${TESTVECS_INSTALL_DIR}/models
> >> + cp ${WORKDIR}/tfmodel/mobilenet_v1_1.0_224_frozen.pb
> >${D}${TESTVECS_INSTALL_DIR}/models
> >> + cp ${WORKDIR}/git/tests/TfMobileNet-Armnn/labels.txt
> >${D}${TESTVECS_INSTALL_DIR}/models
> >> + install -d ${D}${TESTVECS_INSTALL_DIR}/testvecs
> >> + cp ${WORKDIR}/tidl-api/examples/classification/clips/*.mp4
> >${D}${TESTVECS_INSTALL_DIR}/testvecs
> >> + cp ${WORKDIR}/tidl-api/examples/classification/images/*.jpg
> >> + ${D}${TESTVECS_INSTALL_DIR}/testvecs
> >> chrpath -d ${D}${bindir}/*
> >> }
> >>
> >> CXXFLAGS += "-fopenmp"
> >> LIBS += "-larmpl_lp64_mp"
> >>
> >> -SOLIBS = ".so"
> >> -FILES_SOLIBSDEV = ""
> >> -FILES_${PN} += "{bindir}/*"
> >> -FILES_${PN} += "{libdir}/*"
> >> +FILES_${PN} += "${TESTVECS_INSTALL_DIR}"
> >> FILES_${PN}-dev += "{libdir}/cmake/*"
> >> INSANE_SKIP_${PN}-dev = "dev-elf"
> >> --
> >> 1.9.1
> >>
> >> _______________________________________________
> >> meta-arago mailing list
> >> [email protected]
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
