linorobot / linorobot2

Autonomous mobile robots (2WD, 4WD, Mecanum Drive)
Apache License 2.0
436 stars 148 forks source link

linorobot2 bringup issue on Ubuntumate22.04.3 #85

Open hemsinghb opened 9 months ago

hemsinghb commented 9 months ago

Dear Sir, when I run below code its throwing error as shown below'. However there is no error while executing slam and navigation files, please help me. log file is showing same content which is shown below. I think the issue may be with default_robot.launch or custom_robot.launch . this para of script have some issue with latest kernel...

""line 79-85 in the "linorobt2_bringup.launch.py"

from launch.actions import DeclareLaunchArgument, IncludeLaunchDescription from launch.substitutions import LaunchConfiguration, PathJoinSubstitution, PythonExpression

    IncludeLaunchDescription(
        PythonLaunchDescriptionSource(default_robot_launch_path),
        condition=UnlessCondition(LaunchConfiguration("custom_robot")),
        launch_arguments={
            'base_serial_port': LaunchConfiguration("base_serial_port")
        }.items()
    ),

""

Thank you,. ... HEM.

rock@rock:~/linorobot2_ws $ ros2 launch linorobot2_bringup bringup.launch.py

[INFO] [launch]: All log files can be found below /home/rock/.ros/log/2023-09-19-22-16-15-735043-rock-48883 [INFO] [launch]: Default logging verbosity is set to INFO [ERROR] [launch]: Caught exception in launch (see debug for traceback): <launch.substitutions.launch_configuration.LaunchConfiguration object at 0x7ffb44442290>

RicardoSPeres commented 9 months ago

Something similar is happening to me, only if I set LINOROBOT2_DEPTH_SENSOR to 'realsense'. I get:

ros2 launch linorobot2_bringup bringup.launch.py
[INFO] [launch]: All log files can be found below /home/peres/.ros/log/2023-09-19-23-46-31-783276-novabot-7236
[INFO] [launch]: Default logging verbosity is set to INFO
[ERROR] [launch]: Caught exception in launch (see debug for traceback): <launch.substitutions.launch_configuration.LaunchConfiguration object at 0xffffad6cb0d0>

If I leave LINOROBOT2_DEPTH_SENSOR blank and use only a laser sensor with LINOROBOT2_LASER_SENSOR (not the realsense), it works fine

hemsinghb commented 9 months ago

Dear Juan Jimeno sir,

Alternatively: Can we run bringup without real sense (by disabling " realsense in .bashrc file) in one terminal and separate realsense launcher in another terminal . OR is there any way to rectify the existing bringup.launch . Please reply. THX ...---Hems.

grassjelly commented 9 months ago

can you try setting LINOROBOT2_DEPTH_SENSOR=“” ?

hemsinghb commented 9 months ago

if i put depth sensor ="" its working with out realsense... no issue .

BUT PROBLEM SOLVED: presently (HUMBLE) the linorobo2_ws> src>launch etc : linorobotbringup whole directory have some issues. so I copied it from galactic distro. its working. It may be right or wrong, Im not sure. But the humble version need relook. Its installation bash file got some issue at line no 150 to 165. " space " issue before flower bracket. Navigation.launch.py file also some issue of adding of static transform_publisher etc.

with regards, HEMS.

hemsinghb commented 9 months ago

Actual problem is located. Its due to addition of oakd... camera in "depth.launch.py" . Problem solved by commenting few lines of code which are related to oakd.