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

jazzy対応(MoveitConfigsBuilder) #78

Merged
merged 62 commits into from
Dec 20, 2024
Merged

jazzy対応(MoveitConfigsBuilder) #78

merged 62 commits into from
Dec 20, 2024

Conversation

mizonon
Copy link
Contributor

@mizonon mizonon commented Aug 8, 2024

What does this implement/fix?

MoveitConfigsBuilderについてのJazzy対応です。

Does this close any currently open issues?

How has this been tested?

crane_prus_exampleのdemo.launchとサンプルプログラムを使って、humble環境と同じように動作するか確認しました。

Any other comments?

誤ってhumble_develブランチから本作業ブランチを切っていたため、改めてmasterブランチから作業ブランチを作成し、複数あったコミットを一つに集約した上でこちらのブランチにコピーし、PR作成しております。

Checklists

* MoveitCofigsBuilderを使ってmoveit_configの枠を作る

* planning_plugins, requesr_adaptorsの設定をompl_planning.yamlの先頭で行う

* ompl_planning.yamlにstart_state_max_bounds_errorの設定を追加

* 7/19ここまで

* MoveitConfigsBuilderを使うに伴った、yamlファイルの指定の項目追加

* 一通りMoveItConfigsBuilderに対応

* MoveIt Setup Assistantを使って設定ファイルを生成

* urdfの指定を追加

* crane_plus_configのrun_move_group.launchを呼ぶ

* Unable to parse SRDFのエラーが解決できない

* demo.launchが起動できるようになった

* 余計なコメントアウト部分削除

* 余計なコメントアウト削除

* rviz上でMottionPlanningができるようになった

* 余計なコメントアウト削除
@mizonon mizonon requested a review from Kuwamai August 8, 2024 06:04
Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

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

動作確認は行っていないのですが、コードの構成等についてコメントしました。ご確認お願いします。

.gitignore Outdated Show resolved Hide resolved
.docker/README.md Outdated Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_moveit_config/config/joint_limits.yaml Outdated Show resolved Hide resolved
crane_plus_moveit_config/package.xml Outdated Show resolved Hide resolved
crane_plus_moveit_config/package.xml Outdated Show resolved Hide resolved
crane_plus_moveit_config/package.xml Outdated Show resolved Hide resolved
crane_plus_moveit_config/launch/run_move_group.rviz Outdated Show resolved Hide resolved
@Kuwamai
Copy link
Contributor

Kuwamai commented Aug 14, 2024

CIに関してはSciurus17と同様に、こちらのenv内のhumbleをjazzyに変更お願いします
https://github.com/rt-net/sciurus17_ros/blob/e96d86a6d0f9097b55ea9a1461aa38f510cde478/.github/workflows/industrial_ci.yml#L20

Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

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

主にSciurus17との共通化についてコメントしました。
まだ動作確認は行っていません。

.gitignore Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_moveit_config/config/ompl_planning.yaml Outdated Show resolved Hide resolved
crane_plus_moveit_config/package.xml Outdated Show resolved Hide resolved
crane_plus_moveit_config/package.xml Outdated Show resolved Hide resolved
crane_plus_moveit_config/launch/run_move_group.launch.py Outdated Show resolved Hide resolved
crane_plus_moveit_config/launch/run_move_group.launch.py Outdated Show resolved Hide resolved
@Kuwamai Kuwamai added the Type: Feature New Feature label Sep 17, 2024
@mizonon
Copy link
Contributor Author

mizonon commented Nov 27, 2024

Gazebo Harmonicへの対応を行いましたので、追加レビューをお願いいたします。

mizonon and others added 6 commits November 29, 2024 19:10
* カメラ追加

* use_sim_timeオプション追加

* カメラの画角調整

* 色検出サンプルの色しきい値調整

* 把持位置調整

* 現バージョンのデフォルト設定を反映

* ArUcoマーカ付きの立方体を追加

* ワールド、launchファイル追加

* use_cameraの有効化

* use_cameraを引数で切り替えられるように変更

* カメラスタンドのURDF修正
@mizonon
Copy link
Contributor Author

mizonon commented Dec 5, 2024

以下の2点について対応しました

  • MoveitConfigsBuilderで設定を行った上でrvizの設定ファイル名を追加のパラメータとして設定する
    -> MoveitConfigsBuilderを使用してMoveItの設定を行いつつ、rvizの設定ファイルをlaunchファイルに渡すような処理を実装しました。

  • Cameraモデル追加に伴うGazeboのign->gz対応
    -> masterブランチから取り込んだGazeboのCameraモデル追加対応について、Jazzyで使用できるように、ign->gzの対応を行いました。

Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

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

コメントしました。ご確認お願いします。
コンフリクトが発生しておりマージできないため、まずはコンフリクトの解消から進めていただきたいです。

crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_gazebo/launch/crane_plus_with_table.launch.py Outdated Show resolved Hide resolved
crane_plus_gazebo/launch/crane_plus_with_table.launch.py Outdated Show resolved Hide resolved
crane_plus_moveit_config/launch/run_move_group.launch.py Outdated Show resolved Hide resolved
Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

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

@mizonon
こちらの変更でrviz_configが切り替えられそうです。

crane_plus_examples/launch/demo.launch.py Outdated Show resolved Hide resolved
crane_plus_moveit_config/launch/run_move_group.launch.py Outdated Show resolved Hide resolved
@mizonon
Copy link
Contributor Author

mizonon commented Dec 13, 2024

コンフリクト解消の対応を行いました。
申し訳ないのですが、CIエラーの対応はお願いいたします

@Kuwamai
Copy link
Contributor

Kuwamai commented Dec 20, 2024

demo.launch.py実行時に警告が出ています。 原因はわかりませんが一旦記録として残します。

WARNING:root:Cannot infer URDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.urdf
WARNING:root:Cannot infer SRDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.srdf
WARNING:root:Cannot infer URDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.urdf
WARNING:root:Cannot infer SRDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.srdf
WARNING:root:Cannot infer URDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.urdf
WARNING:root:Cannot infer SRDF from `/home/rt-user/ros2_ws/install/crane_plus_moveit_config/share/crane_plus_moveit_config`. -- using config/crane_plus.srdf

こちら.seetup_assistantがinstallされていないためでした。

install(FILES .setup_assistant DESTINATION share/${PROJECT_NAME})

Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

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

LGTMです。

@Kuwamai Kuwamai merged commit 478e9a3 into master Dec 20, 2024
2 checks passed
@Kuwamai Kuwamai deleted the support_jazzy branch December 20, 2024 07:15
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