Skip to content

Replace robot_localization with FusionCore#45

Draft
jkaflik wants to merge 15 commits into
mainfrom
wip/fusioncore-localization
Draft

Replace robot_localization with FusionCore#45
jkaflik wants to merge 15 commits into
mainfrom
wip/fusioncore-localization

Conversation

@jkaflik
Copy link
Copy Markdown
Owner

@jkaflik jkaflik commented May 25, 2026

WIP

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ffd5a7b8-1867-46c6-b327-4e775fb1eff1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wip/fusioncore-localization

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@manankharwar
Copy link
Copy Markdown

Hey, noticed you're replacing robot_localization with FusionCore in OpenMowerNext. Really cool to see. The failing check looks like a Docker registry permission issue on the org side, not a code problem. Unit tests and the colcon build are both passing. Happy to help if you hit anything on the FusionCore side while finishing this up.

@jkaflik
Copy link
Copy Markdown
Owner Author

jkaflik commented May 26, 2026

@manankharwar this is WIP. I'm doing some exploration on this topic.

Overall, this project is really ad-hoc hobby work to me - I will be investing more into it upcoming days.

@manankharwar
Copy link
Copy Markdown

Makes sense. No rush at all. I'll be watching the PR if anything comes up.

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