Skip to content

Commit

Permalink
feat: conformed srr520 to the standard launchers and updated document…
Browse files Browse the repository at this point in the history
…ation

Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]>
  • Loading branch information
knzo25 committed Mar 14, 2024
1 parent c5823da commit f49a8d7
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 61 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,7 @@ Supported models, where sensor_model is the ROS param to be used at launch:
| Velodyne | VLP-32 | VLP32 | VLP32.yaml | :warning: |
| Velodyne | VLS-128 | VLS128 | VLS128.yaml | :warning: |
| Continental | ARS548 | ARS548 | ARS548.yaml | :warning: |
| Continental | SRR520 | SRR520 | SRR520.yaml | :warning: |

Test status:\
:heavy_check_mark:: complete\
Expand Down
1 change: 1 addition & 0 deletions nebula_ros/config/continental/SRR520.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,4 @@
new_plug_bottom: true
new_cover_damping: 0.0
reset_sensor_configuration: false
configuration_vehicle_wheelbase: 2.79
6 changes: 0 additions & 6 deletions nebula_ros/launch/continental_launch_all_hw.xml
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,6 @@
<param name="configuration_vehicle_width" value="$(var configuration_vehicle_width)"/>
<param name="configuration_vehicle_height" value="$(var configuration_vehicle_height)"/>
<param name="configuration_vehicle_wheelbase" value="$(var configuration_vehicle_wheelbase)"/>

<param name="use_sim_time" value="false"/>
</node>
</group>

Expand Down Expand Up @@ -112,8 +110,6 @@
<param name="configuration_host_port" value="$(var configuration_host_port)"/>
<param name="configuration_sensor_port" value="$(var configuration_sensor_port)"/>
<param name="use_sensor_time" value="$(var use_sensor_time)"/>

<param name="use_sim_time" value="false"/>
</node>
</group>

Expand Down Expand Up @@ -150,8 +146,6 @@
<param name="filters" value="$(var filters)"/>
<param name="use_bus_time" value="$(var use_bus_time)"/>
<param name="configuration_vehicle_wheelbase" value="$(var configuration_vehicle_wheelbase)"/>

<param name="use_sim_time" value="false"/>
</node>
</group>
</launch>
51 changes: 0 additions & 51 deletions nebula_ros/launch/continental_launch_all_hw_rdv.xml

This file was deleted.

6 changes: 2 additions & 4 deletions nebula_ros/launch/nebula_launch.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,17 +22,15 @@ def get_sensor_make(sensor_name):
return "Velodyne", ".yaml"
elif sensor_name.lower() in ["helios", "bpearl"]:
return "Robosense", None
elif sensor_name.lower() == "ars548":
elif sensor_name.lower() in ["ars548", "srr520"]:
return "Continental", None
return "unrecognized_sensor_model", None

def get_plugin_name(sensor_make, sensor_model):
if sensor_make.lower() != "continental":
return sensor_make
elif sensor_model.lower() == "ars548":
return "ContinentalARS548"
else:
return "invalid_plugin"
return sensor_make + sensor_model

def is_hw_monitor_available(sensor_make):
return sensor_make.lower() != "continental"
Expand Down

0 comments on commit f49a8d7

Please sign in to comment.