Skip to content
This repository was archived by the owner on Sep 16, 2024. It is now read-only.

Enable Video Analytic Gstreamer Plugins on Xeon and XeonE3#41

Merged
dpatel257 merged 9 commits intoOpenVisualCloud:masterfrom
fkhoshne:video-analytics-gst
Mar 27, 2019
Merged

Enable Video Analytic Gstreamer Plugins on Xeon and XeonE3#41
dpatel257 merged 9 commits intoOpenVisualCloud:masterfrom
fkhoshne:video-analytics-gst

Conversation

@fkhoshne
Copy link
Copy Markdown
Contributor

No description provided.

@fkhoshne fkhoshne self-assigned this Mar 20, 2019
@fkhoshne fkhoshne requested review from dpatel257 and xwu2git March 20, 2019 17:50
@jhou5
Copy link
Copy Markdown
Contributor

jhou5 commented Mar 20, 2019

Seen from the code, the pull request are for 2 PRs:

  1. Need Gstreamer inference engine plugins Need Gstreamer inference engine plugins #21
  2. Request to DLDT MO in the dev pkg Request to DLDT MO in the dev pkg #24

Am I right?

@fkhoshne
Copy link
Copy Markdown
Contributor Author

Seen from the code, the pull request are for 2 PRs:

  1. Need Gstreamer inference engine plugins Need Gstreamer inference engine plugins #21
  2. Request to DLDT MO in the dev pkg Request to DLDT MO in the dev pkg #24

Am I right?

Yes. It includes the PR for the gstreamer inference engine plugins and the PR for DLDT MO in the dev pkg. I've put the MO code inside the dldt-ie.m4 template (since IE and MO are from the same repo, it makes more sense to me that they share the same template instead of a dldt-ie and dldt-mo template). With that said, the dldt-ie.m4 should maybe be renamed to dldt.m4

@dpatel257
Copy link
Copy Markdown

ffmpeg_dldt_gst_dev package for centos75 with following error. related to how dldt is configured:

Step 126/126 : RUN cd /home/FFmpeg && ./configure --prefix="/usr" --libdir=/usr/lib64 --extra-libs="-lpthread -lm" --enable-shared --enable-gpl --enable-¦·······························
libass --enable-libfreetype --enable-openssl --enable-nonfree --disable-vaapi --disable-hwaccels --enable-libfdk-aac --enable-libmp3lame --enable-libopus -¦·······························
-enable-libvorbis --enable-libvpx --enable-libx264 --enable-libx265 --enable-libaom --enable-libsvthevc --enable-libsvtav1 --enable-libinference_engine && make -j8 && make install DESTDIR="/home/build"
---> Running in 14c91260b187
ERROR: dldt not found using pkg-config

If you think configure made a mistake, make sure you are using the latest
version from Git. If the latest version fails, report the problem to the
ffmpeg-user@ffmpeg.org mailing list or IRC #ffmpeg on irc.freenode.net.
Include the log file "ffbuild/config.log" produced by configure as this will help solve the problem.

The command '/bin/sh -c cd /home/FFmpeg && ./configure --prefix="/usr" --libdir=/usr/lib64 --extra-libs="-lpthread -lm" --enable-shared --enable-gpl --enable-libass --enable-libfreetype --enable-openssl --enable-nonfree --disable-vaapi --disable-hwaccels --enable-libfdk-aac --enable-libmp3lame --enable-libopus --enable-libvorbis --enable-libvpx --enable-libx264 --enable-libx265 --enable-libaom --enable-libsvthevc --enable-libsvtav1 --enable-libinference_engine && make -j8 && make install DESTDIR="/home/build"' returned a non-zero code: 1

Xeon/centos-7.5/ffmpeg+gst+dev/CMakeFiles/build_xeon_ffmpeg_gst_devel_centos75.dir/build.make:57: recipe for target 'Xeon/centos-7.5/ffmpeg+gst+dev/CMakeFiles/build_xeon_ffmpeg_gst_devel_centos75' failed
make[2]: *** [Xeon/centos-7.5/ffmpeg+gst+dev/CMakeFiles/build_xeon_ffmpeg_gst_devel_centos75] Error 1
CMakeFiles/Makefile2:2517: recipe for target 'Xeon/centos-7.5/ffmpeg+gst+dev/CMakeFiles/build_xeon_ffmpeg_gst_devel_centos75.dir/all' failed
make[1]: *** [Xeon/centos-7.5/ffmpeg+gst+dev/CMakeFiles/build_xeon_ffmpeg_gst_devel_centos75.dir/all] Error 2
Makefile:94: recipe for target 'all' failed
make: *** [all] Error 2

Copy link
Copy Markdown
Contributor

@xwu2git xwu2git left a comment

Choose a reason for hiding this comment

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

Please add:
(1) update test status in readme.md.
(2) add typical use case examples in doc/gst.md
(3) add test cases.

@dpatel257
Copy link
Copy Markdown

All tested passed. Will merge

@dpatel257 dpatel257 merged commit 4615ce3 into OpenVisualCloud:master Mar 27, 2019
lylintel pushed a commit that referenced this pull request Jun 16, 2023
Fix hadolint warning SC2164: Use cd ... || exit in case cd fails.
lylintel pushed a commit that referenced this pull request Jun 16, 2023
Fix hadolint warning SC2164: Use cd ... || exit in case cd fails.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants