Skip to content

Feature/robot self filter#123

Open
ashvinganesan wants to merge 6 commits into
feature/nav2-tf-fixfrom
feature/robot-self-filter
Open

Feature/robot self filter#123
ashvinganesan wants to merge 6 commits into
feature/nav2-tf-fixfrom
feature/robot-self-filter

Conversation

@ashvinganesan

@ashvinganesan ashvinganesan commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

This PR adds the robot-self filter package. This means that the robot doesn't see itself and think that it is hitting a wall/object. #107

@ashvinganesan ashvinganesan marked this pull request as ready for review April 10, 2026 21:14

@TagCiccone TagCiccone left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When building robot_self_filter with colcon, a dev warning comes up. It's not an error, so we can supress it.

In colcon.meta: Add

robot_self_filter:
    cmake-args:
        - "-DCMAKE_POLICY_DEFAULT_CMP0144=NEW"

to the end of the file.

(colcon.meta exists on /develop, so either merge develop into this branch or copy it wholesale)

@ashvinganesan

Copy link
Copy Markdown
Contributor Author

When building robot_self_filter with colcon, a dev warning comes up. It's not an error, so we can supress it.

In colcon.meta: Add

robot_self_filter:
    cmake-args:
        - "-DCMAKE_POLICY_DEFAULT_CMP0144=NEW"

to the end of the file.

(colcon.meta exists on /develop, so either merge develop into this branch or copy it wholesale)

copied the file and added the change.

@ashvinganesan ashvinganesan dismissed TagCiccone’s stale review April 21, 2026 20:41

still showing up as requesting change even when I do rerequest (not sure why) I made the changes

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.

3 participants