Skip to content

Update ekf.yaml and display.launch.py#109

Open
coffinfish wants to merge 1 commit intoros-navigation:rollingfrom
coffinfish:rolling
Open

Update ekf.yaml and display.launch.py#109
coffinfish wants to merge 1 commit intoros-navigation:rollingfrom
coffinfish:rolling

Conversation

@coffinfish
Copy link

@coffinfish coffinfish commented Mar 5, 2025

Changed ekf_node to ekf_filter_node for consistency as per #650

Sorry for all the force-pushes, I am still learning git.

Changed ekf_node to ekf_filter_node for consistency

Signed-off-by: seatoad <cindysongh@gmail.com>
Copy link
Member

@SteveMacenski SteveMacenski left a comment

Choose a reason for hiding this comment

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

Missed two in this file:

Copy link
Collaborator

@mini-1235 mini-1235 left a comment

Choose a reason for hiding this comment

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

I think the executable should still be ekf_node so I am approving this PR

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