-
Notifications
You must be signed in to change notification settings - Fork 0
Update realsense package #316
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
base: main
Are you sure you want to change the base?
Update realsense package #316
Conversation
7b923ec to
8c63232
Compare
| set -e | ||
| source /home/$UNAME/.bashrc | ||
| # apt update | ||
| apt-get update # && apt-get upgrade -y && apt-get dist-upgrade -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would remove any unneeded, commented-out code (very nitpicky I know)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Lets test it tomorrow on at least one other laptop than yours and preferably on the Panther computer and merge afterwards.
…2 package to core_packages Signed-off-by: Rosalie <[email protected]>
…se ros wrapper, fixed it now Signed-off-by: Rosalie <[email protected]>
Signed-off-by: Rosalie <[email protected]>
Signed-off-by: Rosalie <[email protected]>
Signed-off-by: Rosalie <[email protected]>
…ic names Signed-off-by: Rosalie <[email protected]>
Signed-off-by: Rosalie <[email protected]>
Signed-off-by: Rosalie <[email protected]>
…messages Signed-off-by: Rosalie <[email protected]>
8c63232 to
3febc87
Compare
Description
The realsenses-ros wrapper was updated to also allow for adding TF prefixes, but this version is not yet available as a ros package. This required an install from source.
v4.57.2is chosen since this is the release that contains the TF prefix functionality, the newer versionv4.57.3caused crashes during the building phase.Consequently, the RealSense SDK also required a newer version than currently available as ros package, and also required an install from source.
Fixes: #206
Testing
Tested the real camera via the laptop using:
pytest ros2_ws/src/rcdt_test/rcdt_test/sensors/integration/test_realsense.py --simulation FalseDocumentation
Additional Notes
When running a tool like
rs-enumerate-devices -sthe command line provides a small error but proceeds to work properly:ERROR [136136118261632] (context.cpp:40) No valid configuration file found at : /home/rosalie/.realsense-config.json loading defaults Device Name Serial Number Firmware Version Intel RealSense D435I 923322072624 5.17.0.10We will ignore this for now since it is not relevant for our pipeline.