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

Tx2 install #195

Closed
wants to merge 43 commits into from
Closed

Tx2 install #195

wants to merge 43 commits into from

Conversation

Pavlos-Tosidis
Copy link
Collaborator

TX-2 installation script.
Updated installation.md and fixed some import statements in object_detection_2d/detr and object_detection_2d/gem

Updated installation.md and changed import statements in object_detection_2d gem and detr.
@Pavlos-Tosidis Pavlos-Tosidis added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 10, 2022
@ad-daniel ad-daniel added test sources Run style checks test tools Test the toolkit methods labels Jan 11, 2022
@ad-daniel
Copy link
Collaborator

I don't have a TX2 to test this, can somebody else validate that the script works? @thomaspeyrucain maybe?

bin/install_tx2.sh Outdated Show resolved Hide resolved
@thomaspeyrucain
Copy link
Contributor

I don't have a TX2 to test this, can somebody else validate that the script works? @thomaspeyrucain maybe?

@ad-daniel Ok I will test it on a docker on our Jetson

Tested it twice on two different TX2 boards successfully.
Copy link
Collaborator

@ad-daniel ad-daniel left a comment

Choose a reason for hiding this comment

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

Thank you, just a small possible improvement.
As for whether it works or not, as I said I can't really test it.

bin/install_tx2.sh Outdated Show resolved Hide resolved
bin/install_tx2.sh Outdated Show resolved Hide resolved
bin/install_tx2.sh Outdated Show resolved Hide resolved
docs/reference/installation.md Outdated Show resolved Hide resolved
docs/reference/installation.md Outdated Show resolved Hide resolved
Pavlos-Tosidis and others added 6 commits March 14, 2022 16:06
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
@Pavlos-Tosidis
Copy link
Collaborator Author

@Pavlos-Tosidis I discussed this with my colleague and we checked the log, it is missing some things inside the docker like cudnn.h and libcurand.so.10 could you please check that everything is correctly installed in your docker?

@thomaspeyrucain as we discussed, I have flashed and installed the docker image multiple times on a TX2, with the results being the same. I didn't try the ROS scripts, but the python scripts run successfully.

@thomaspeyrucain
Copy link
Contributor

Did you try on any NX boards?

@thomaspeyrucain
Copy link
Contributor

thomaspeyrucain commented May 20, 2022

Do you build the docker image from scratch ?
This base image : nvcr.io/nvidia/l4t-base:r32.6.1
Does not contain the cudnn.h and the corresponding libraries

@Pavlos-Tosidis
Copy link
Collaborator Author

Do you build the docker image from scratch ? This base image : nvcr.io/nvidia/l4t-base:r32.6.1 Does not contain the cudnn.h and the corresponding libraries

Do you run the container with the proper arguments?
-it --privileged

@thomaspeyrucain
Copy link
Contributor

The errors are occurring when building the docker from the Dockerfile
Those arguments -it --privileged is when you run the docker image

@Pavlos-Tosidis
Copy link
Collaborator Author

The errors you sent me so far are while trying to import torch/mxnet.
There are a lot of warnings when building mxnet, but it gets installed and working as intended.

@thomaspeyrucain
Copy link
Contributor

It showed some errors during the process but it finished building the image OpenDR_docker_build_NX.txt

If you check the logfile errors are appearing when building the docker image

@Pavlos-Tosidis
Copy link
Collaborator Author

It showed some errors during the process but it finished building the image OpenDR_docker_build_NX.txt

If you check the logfile errors are appearing when building the docker image

Since it produces the docker image in the end. Can you please try and run for example (after activating opendr):
python3 projects/perception/face_recognition/demos/infrence_demo.py

@Pavlos-Tosidis
Copy link
Collaborator Author

@thomaspeyrucain Just finished the docker image installation on an Nvidia NX. Everything run smoothly and tried face_recognition and Retinaface inference demos for testing. Both run as intended.

@thomaspeyrucain
Copy link
Contributor

thomaspeyrucain commented May 25, 2022

@Pavlos-Tosidis Nice, Did you succeed to run the ROS nodes without issues inside the docker?

@Pavlos-Tosidis
Copy link
Collaborator Author

NX docker image is up on docker hub.
you can download and run it with:
xhost +local:root
sudo docker run -it --privileged -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=unix$DISPLAY opendr/opendr-toolkit:nx /bin/bash

Fixed a couple of ROS issues.
If the NX is correctly flashed (with Cuda and nvidia runtime) it works out of the box

@thomaspeyrucain
Copy link
Contributor

I pulled the image and tested it on the NX board and it works for the face recognition demo script
What ROS issues did you fix?

@ad-daniel
Copy link
Collaborator

The broken tests on macos can be ignored, cppcheck was upgraded and it's being fixed on a separate PR

@ad-daniel
Copy link
Collaborator

#360 is now included in develop

@ad-daniel
Copy link
Collaborator

#317 is now included in develop

@Pavlos-Tosidis Pavlos-Tosidis mentioned this pull request Dec 20, 2022
@tsampazk
Copy link
Collaborator

Replaced by #384

@tsampazk tsampazk closed this Dec 20, 2022
@Pavlos-Tosidis Pavlos-Tosidis deleted the tx2_install branch December 23, 2022 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request test sources Run style checks test tools Test the toolkit methods
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants