Skip to content

Update plugins to new sensors API#61

Merged
fmrico merged 3 commits intoEasyNavigation:rollingfrom
Butakus:rolling
Apr 3, 2026
Merged

Update plugins to new sensors API#61
fmrico merged 3 commits intoEasyNavigation:rollingfrom
Butakus:rolling

Conversation

@Butakus
Copy link
Copy Markdown
Member

@Butakus Butakus commented Mar 31, 2026

This PR adapts all plugins to the new API after the refactor in EasyNavigation/EasyNavigation#94.

  • The paths have changed: Perception classes now live under easynav_sensors instead of easynav_common.
  • Most plugins used the "points" NavState key as a way to get all point clouds. Now sensors live independently of groups. I updated the code to get the list of points from the "points" group, so the rest of the logic is the same. The new NavState API ofr groups returns a vector of perceptions the same way the point clouds were stored in the "points" key, so the change is minimal. Beware that the group key must be explicitly set in the YAML config file, as it is not set by default.

Please do not merge until and unless the core PR is approved!

@fmrico
Copy link
Copy Markdown
Contributor

fmrico commented Apr 2, 2026

I have more changes here. I will do it as soon as #94 is finished

fmrico added 2 commits April 3, 2026 10:17
Signed-off-by: Francisco Martín Rico <fmrico@gmail.com>
Signed-off-by: Francisco Martín Rico <fmrico@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@fmrico fmrico merged commit 7cb2969 into EasyNavigation:rolling Apr 3, 2026
2 checks passed
@fmrico
Copy link
Copy Markdown
Contributor

fmrico commented Apr 3, 2026

merging!! 🚀

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.

2 participants