Humble対応 #6
Merged
Humble対応 #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
SLAMとNavigationが正常に動作することを確認しました。 |
|
CIがfoxyのままだと思うので、humbleへの変更をお願いします（現在はCIが動いていませんが、変更をpushしたら動くと思います）。 |
|
raspimouse_ros2_examplesにhumble-develブランチを作成しました。 |
ShotaAk
reviewed
Aug 4, 2023
| @@ -417,7 +417,7 @@ Visualization Manager: | |||
| Global Options: | |||
| Background Color: 48; 48; 48 | |||
| Fixed Frame: map | |||
| Frame Rate: 30 | |||
| Frame Rate: 5 | |||
There was a problem hiding this comment.
Frame Rateを変更した理由をPRのコメントに書き残してください。
README.md
Outdated
| @@ -7,6 +7,7 @@ Raspberry Pi MouseでSLAMが行える`raspimouse_slam`パッケージとナビ | |||
|
|
|||
| 現在、以下のROS 2のディストリビューションに対応しております。 | |||
| - Foxy ([foxy-devel](https://github.com/rt-net/raspimouse_slam_navigation_ros2/tree/foxy-devel)) | |||
| - Humble ([TBD]()) | |||
There was a problem hiding this comment.
リンクわかる（予測できる）ので今のうちに追加しちゃってもいい気がしてきましたが、どうでしょう？
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this implement/fix?
ROS 2 Humbleに対応するように、Launchファイルの書き換えなどを行いました。
Does this close any currently open issues?
#5
How has this been tested?
READMEのQuick Startで記述したSLAMとナビゲーションを、実機で行い動作確認をしました。
Any other comments?
現時点では、raspimouse_ros2_examplesに
humble-develブランチがないため、READMEに記載されているインストール方法だと失敗します。（raspimouse_slam_navigation_ros2も同様です）どちらも
mainブランチを指定するようにしておくのが妥当でしょうか？→
humble-develブランチが作成されたので解決しました。RVizファイルのFrame Rateを30→5Hzに変更しているのは、LiDAR情報を表示するための変更です。
（参考: https://answers.ros.org/question/413915/rviz2-lookup-would-require-extrapolation-into-the-future/ ）
Checklists