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

ライントレース用のコースを作成 #74

Merged
merged 14 commits into from
Nov 17, 2023
Merged

Conversation

YusukeKato
Copy link
Contributor

@YusukeKato YusukeKato commented Nov 15, 2023

What does this implement/fix?

ライントレース用コースのワールドを作成しました。

Does this close any currently open issues?

しません。

How has this been tested?

下記のコマンドを実行するとライントレース用サンプルコースのワールドが表示されます。また、RGBカメラを搭載したRaspberry Pi Mouseも配置されます。

ros2 launch raspimouse_gazebo raspimouse_with_line_follower_field.launch.py use_rgb_camera:=true

Any other comments?

Checklists

@YusukeKato YusukeKato added the Type: Feature New Feature label Nov 15, 2023
@YusukeKato YusukeKato self-assigned this Nov 15, 2023
@YusukeKato YusukeKato marked this pull request as ready for review November 15, 2023 08:29
Copy link

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

コメントしました。
拡張性が高いサンプルなので今のうちに準備しておきたいです。対応お願いします。

Comment on lines 4 to 5
<physics name="1ms" type="ignored">
<max_step_size>0.001</max_step_size>
Copy link

Choose a reason for hiding this comment

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

シミュレーション速度が50%を切ることがあるので、動作周期を小さくしてください。

Suggested change
<physics name="1ms" type="ignored">
<max_step_size>0.001</max_step_size>
<physics name="10ms" type="ignored">
<max_step_size>0.010</max_step_size>

Copy link

Choose a reason for hiding this comment

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

他のworldファイルも変更したほうがようです。これは別PRで対応お願いします。

world_launch = IncludeLaunchDescription(
PythonLaunchDescriptionSource([
get_package_share_directory('raspimouse_gazebo'),
'/launch/raspimouse_with_emptyworld.launch.py']),
Copy link

Choose a reason for hiding this comment

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

ラズパイマウスをspawnする位置を変更できるように、emptyworld.launch.pyに引数を追加してください。
参考情報:https://github.com/rt-net/crane_x7_ros/blob/deaa39a9c821b975b450597d62471c5d8f054c93/crane_x7_gazebo/launch/crane_x7_with_table.launch.py#L50

まずは、回転なし、位置のみ(x, y, z)でお願いします。

line_follower_filed.launch.pyには引数無くてokです。

例:

    world_launch = IncludeLaunchDescription(
        PythonLaunchDescriptionSource([
            get_package_share_directory('raspimouse_gazebo'),
            '/launch/raspimouse_with_emptyworld.launch.py']),
        launch_arguments={
            'world_name': world_file,
            'spawn_x': 0.0
            'spawn_y': 0.0
            'spawn_z': 0.1
            }.items()
    )

Copy link

Choose a reason for hiding this comment

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

ライントレースフィールドをいじり始めると、ロボットの初期位置を変えたくなるので。

Comment on lines 9 to 10
<name>Atsushi Kuwagata</name>
<email>[email protected]</email>
Copy link

Choose a reason for hiding this comment

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

名前変更お願いします

<COLLADA xmlns="http://www.collada.org/2005/11/COLLADASchema" version="1.4.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
<asset>
<contributor>
<author>Blender User</author>
Copy link

Choose a reason for hiding this comment

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

AuthorはKatoさんの名前を入れてください。

Copy link

Choose a reason for hiding this comment

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

直線のデータも同様にお願いします。

<asset>
<contributor>
<author>Blender User</author>
<authoring_tool>Blender 2.92.0 commit date:2021-02-24, commit time:16:25, hash:02948a2cab44</authoring_tool>
Copy link

Choose a reason for hiding this comment

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

blender 4.0がリリースされたのでそれで編集してください。

Copy link

Choose a reason for hiding this comment

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

パネルの寸法がわかるように、course_curve_50x50cm のような名称にしてください。

フォルダ名等も合わせて変更お願いします。

<geometry>
<mesh>
<uri>model://course_curve/meshes/course_curve.dae</uri>
<scale>0.5 0.5 0.5</scale>
Copy link

Choose a reason for hiding this comment

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

scaleを使用しないようにもとのdaeファイルを編集してください。

ライン幅が何cmなのかが分かりにくいためです。

Copy link

Choose a reason for hiding this comment

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

パネルのサイズ、黒線の太さに付いてREADMEに説明を記載してください。

このjpg画像をREADMEに貼ると説明しやすいと思います。

例:

モデルデータ一覧

cource_curve_50x50cm

ライントレース用の曲線コースパネルです。
パネルサイズは50cm x 50cm、直線幅は 〇〇cmです

(ここに画像)

cube**

それぞれ、一辺5cm、7.5cm、10cm、15cmの立方体です

(ここにスクショ)

Copy link

Choose a reason for hiding this comment

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

daeファイルはBlender 4.0で編集しています。のような補足も入れてほしいです。

Copy link

Choose a reason for hiding this comment

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

コースを増やす可能性が高いのでファイル名を柔軟にしたいです。

line_follower_field1.sdf にファイル名変更お願いします

world_file = os.path.join(
get_package_share_directory('raspimouse_gazebo'),
'worlds',
'line_follower_world.sdf')
Copy link

Choose a reason for hiding this comment

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

ファイル名をline_follower_field1.sdfに変更お願いします。

将来的にはfieldのような変数を用意して、
**launch.py field:=2 のようにフィールドを切り替えたいです。

@YusukeKato
Copy link
Contributor Author

ライントレース用コースの線幅を4cmに変更しました。

Copy link

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

LGTMです。

@ShotaAk ShotaAk merged commit 78b5a51 into ros2 Nov 17, 2023
1 check passed
@ShotaAk ShotaAk deleted the add-line-follower-field branch November 17, 2023 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants