dusty-nv / jetson-containers

Machine Learning Containers for NVIDIA Jetson and JetPack-L4T
MIT License
2.18k stars 448 forks source link

Deepstream setup adjustments #409

Closed Mark-Firestorm closed 7 months ago

Mark-Firestorm commented 7 months ago

When the python bindings are cloned they appear to be cloned in the wrong folder for being able to run the examples. I'm not sure of the exact directory that the repo is supposed to be cloned in but it is one level deeper than it is currently.

Proposed fix for this would be on line 65 of the Deepstream dockerfile, changing it to the following:

RUN cd /opt/nvidia/deepstream/deepstream/sources && \

Evidence for this being in the wrong place can be seen in the config for deepstream_test1_usbcam where in the Deepstream folder structure, the sources folder being referenced for the model to be used is one level off in the current Docker environment.

Edit: corrected mistake in the proposed fix

dusty-nv commented 7 months ago

@Mark-Firestorm if you know this fix and could start submitting PRs for these that would be great, I will accept them. Sorry been really busy, and don't have the time at the moment.

Mark-Firestorm commented 7 months ago

@dusty-nv I'll put together a fork and a PR now, thanks!

Mark-Firestorm commented 7 months ago

PR submitted

dusty-nv commented 7 months ago

Thanks Mark! I merged your PR https://github.com/dusty-nv/jetson-containers/pull/411 into dev. From there I will merge it into master 👍

Mark-Firestorm commented 7 months ago

Sure thing, thanks for the quick turn around!