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

Feature/update bringup #30

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

AiriYokochi
Copy link
Contributor

@AiriYokochi AiriYokochi commented Nov 27, 2023

Summary

bringupの改修をしました
IMU.launchを追加しました
melodicのLiDARが前後逆に出力されるバグがnoeticで修正されたので対応しました

Test

  • gazebo環境で起動した
  • 実機環境で起動した

Comment on lines 24 to 31
<!-- launch IMU -->
<!-- <include file="$(find wt901B_ros)/launch/wt901B_ros.launch"/> -->
<!-- <remap from="scan" to="scan" /> -->
<!-- <remap from="scan_filtered" to="scan_filtered"/> -->
<!-- v1 -->
<!-- <rosparam command="load" file="$(find cube_petit_bringup)/config/sensors/cube_petit_pacecat_filter.yaml" /> -->
<!-- v2 -->
<!-- imaha iranai -->
Copy link
Contributor

Choose a reason for hiding this comment

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

消す

Copy link
Contributor Author

Choose a reason for hiding this comment

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

削除しました

Comment on lines 5 to 12
<!-- launch カメラ -->>
<node pkg="uvc_camera" type="uvc_camera_node" name="usb_camera_node">
<param name="device" type="string" value="/dev/video0"/>
<!-- <node pkg="uvc_camera" type="uvc_camera_node" name="usb_camera_node">
<param name="device" type="string" value="/dev/rear_camera"/>
<param name="pixel_format" value="YUYV"/>
<param name="camera_frame_id" value="rear_camera"/>
<param name="io_method" value="mmap"/>
<param name="frame_id" value="rear_camera_optical_link"/>
<param name="camera_info_url" type="string" value="file://$(find cube_petit_bringup)/config/sensors/rear_camera/camera.yaml"/> </node> -->
Copy link
Contributor

Choose a reason for hiding this comment

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

消す

Copy link
Contributor Author

Choose a reason for hiding this comment

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

削除しました

Comment on lines 24 to 34
<!-- <arg name="marker_size" default="5.5" />
<arg name="max_new_marker_error" default="0.08" />
<arg name="max_track_error" default="0.2" />
<arg name="cam_image_topic" default="/rear_camera/image_raw" />
<arg name="cam_info_topic" default="/rear_camera/camera_info" />
<arg name="output_frame" default="/rear_cameras" /> -->

<!-- launch AR_TRACK_ALVAR -->
<!-- <node name="ar_track_alvar" pkg="ar_track_alvar" type="individualMarkersNoKinect" respawn="false" output="screen"
args="$(arg marker_size) $(arg max_new_marker_error) $(arg max_track_error)
args="$(arg marker_size) $(arg max_new_marker_error) $(arg max_track_error)
$(arg cam_image_topic) $(arg cam_info_topic) $(arg output_frame)" /> -->
Copy link
Contributor

Choose a reason for hiding this comment

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

消す

Copy link
Contributor Author

Choose a reason for hiding this comment

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

削除しました

Comment on lines 6 to 8
<!-- version1 -->
<!-- <arg name="model" default="$(find cube_petit_description)/urdf/cube_petit.xacro" /> -->
<!-- <rosparam command="load" file="$(find cube_petit_navigation)/config/odom_gazebo.yaml"/> -->
Copy link
Contributor

Choose a reason for hiding this comment

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

version 1 と2で両方対応したい場合はコメントアウトではなくarg ifで切り替えられるようにしたほうが良いでしょう

Copy link
Contributor Author

Choose a reason for hiding this comment

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

そうします、ありがとうございます

@@ -74,19 +58,17 @@
-->
Copy link
Contributor

Choose a reason for hiding this comment

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

消す

Copy link
Contributor Author

Choose a reason for hiding this comment

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

削除しました

<!-- Charging simulation -->
<include file="$(find cube_petit_gazebo)/launch/include/docking_station_simulation.launch">
<arg name="robot" value="cube-petit"/>
</include>
</launch>
Copy link
Contributor

Choose a reason for hiding this comment

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

改行追加

Copy link
Contributor Author

Choose a reason for hiding this comment

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

追加しました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants