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

deepstream-tao-apps: add recipe #7

Closed
wants to merge 2 commits into from

Conversation

bcis93
Copy link

@bcis93 bcis93 commented Sep 30, 2021

This PR adds a recipe that builds several deepstream TAO sample apps as well as the libnvds_infercustomparser_tao.so custom parser library from NVIDIA's deepstream_tao_apps repository.

I separated the samples and custom parser into separate packages so that the custom parser can be built by itself if desired.

@bcis93
Copy link
Author

bcis93 commented Sep 30, 2021

Note that this recipe doesn't build yet... see details in #9

@ichergui
Copy link
Member

ichergui commented Oct 1, 2021

Hey @bcis93

You can try adding the recipe into dunfell-l4t-r32.4.3 to avoid the incompatibility between TensorRT and DeepStream.

This commit adds a recipe that builds several deepstream TAO sample apps as well as the libnvds_infercustomparser_tao.so custom parser library from NVIDIA's https://github.com/NVIDIA-AI-IOT/deepstream_tao_apps repository.

Signed-off-by: Bryan Cisneros <bcisneros@sighthound.com>
@bcis93 bcis93 changed the base branch from master to dunfell December 3, 2021 01:56
@bcis93 bcis93 marked this pull request as ready for review December 3, 2021 01:57
Copy link
Member

@madisongh madisongh left a comment

Choose a reason for hiding this comment

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

A few changes needed, see in-line comments.

Thanks!

Correct the license to MIT, add the github repo as the homepage, and append the git hash to the version.

Signed-off-by: Bryan Cisneros <bcisneros@sighthound.com>
@madisongh
Copy link
Member

The PV change required a minor fix (dropping the .${PV} suffix from the .so), and the PACKAGES additions needed to be set via prepend rather than append. I took care of these and merged this manually.

@madisongh madisongh closed this Dec 4, 2021
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