Skip to content

Feature/add route server UI editor#684

Merged
SteveMacenski merged 24 commits intoros-navigation:masterfrom
bekirbostanci:feature/add-route-server-ui-editor
May 3, 2025
Merged

Feature/add route server UI editor#684
SteveMacenski merged 24 commits intoros-navigation:masterfrom
bekirbostanci:feature/add-route-server-ui-editor

Conversation

@bekirbostanci
Copy link
Copy Markdown
Contributor

This PR adds a step-by-step guide for generating route graphs using the VDA LIF Editor, a web-based tool for creating maps in Nav2-compatible GeoJSON formats.

The documentation covers:

  • Tool overview and usage
  • Coordinate system setup
  • Node and edge creation
  • Exporting to ROS GeoJSON and LIF

Useful for anyone integrating VDA-compliant maps into ROS-based systems.

https://vda5050-lif-editor.vercel.app/

@bekirbostanci bekirbostanci force-pushed the feature/add-route-server-ui-editor branch from 8195a99 to e0804d4 Compare May 2, 2025 00:21
Copy link
Copy Markdown
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.

Looks great!

Comment thread configuration/packages/configuring-route-server.rst
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst Outdated
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst Outdated
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst Outdated
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst Outdated
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst Outdated
Comment thread tutorials/docs/route_graph_generation_lif_editor.rst
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
…tutorial references

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
…oute Graph Generation LIF Editor documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
…tion

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
… documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
… Route Graph Generation LIF Editor documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
… Editor documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
@bekirbostanci bekirbostanci force-pushed the feature/add-route-server-ui-editor branch from a24e331 to bbc138f Compare May 2, 2025 20:35
… LIF Editor documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
…LIF Editor documentation

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
@bekirbostanci bekirbostanci force-pushed the feature/add-route-server-ui-editor branch from b5c3e57 to 7a003ae Compare May 2, 2025 21:30
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
…e paths

Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
Signed-off-by: bekirbostanci <bekirbostanci@gmail.com>
@bekirbostanci bekirbostanci force-pushed the feature/add-route-server-ui-editor branch from b4f6c06 to 0ea77de Compare May 2, 2025 21:53
Comment thread tutorials/docs/route_server_tools.rst
Comment thread tutorials/docs/route_server_tools.rst
Comment thread tutorials/docs/route_server_tools.rst Outdated
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
@SteveMacenski SteveMacenski merged commit 7117153 into ros-navigation:master May 3, 2025
3 checks passed
@SteveMacenski
Copy link
Copy Markdown
Member

SteveMacenski commented Sep 10, 2025

FYI @bekirbostanci I've tried using this recently and ran into an issue. This looks fine:

image

But that's not what's exported:

image

I can also see if this I export the LIF file and try to open it in a new tab, it also looks crumpled and to the wrong scale (my slam was at 0.02m resolution, the width/height are set correctly). I also see it created 2x edges that were half populated (had startid but no endid) and didn't need to exist at all - which took me quite awhile to figure out.

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

bekirbostanci commented Sep 10, 2025

Hey @SteveMacenski
I see the problem, I will fix it today or tomorrow. Thanks for report.

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

Sorry about the bug and fixed now

@SteveMacenski
Copy link
Copy Markdown
Member

Great! I’m curious what the issue was for my personal curiosity. I tried to check based on manual transcription of a few points and noticed the coordinates actually in the GUI were also crumbled even though they appeared correct.

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

It was entirely my mistake :) The problem was that the network-graph component wasn’t syncing properly with the layout conversion, which affected both the VDA5050 LIF format and ROS network format.

I’ve made some changes, and it should work correctly now. If you notice any other issues, please let me know.

@SteveMacenski
Copy link
Copy Markdown
Member

Does that also explain the couple of half-created edges with a startid but no endid?

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

bekirbostanci commented Sep 11, 2025

No, it doesn’t explain it. By the way, is there only one edge like that and always startid= 0? Because I see another issue: the first point is a connection but also an undefined point. This is reason why there is undefined endid. I am working on that right now.

image image

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

This bug also fixed

@SteveMacenski
Copy link
Copy Markdown
Member

SteveMacenski commented Sep 11, 2025

Oooo nice! What was that one out of curiosity? I didn't see the undefined, it just only had a startid with no endid. I want to say it did have an ID of 0, but I don't recall with certainty. I didn't commit it to my project until after I removed it so the Route Server would load.

Great work by the way, minus these hiccups it was really easy to set up a room for a demo. I made a gif for ROSCon presentation next month

simplescreenrecorder3-2025-09-09_16 35 54

@bekirbostanci
Copy link
Copy Markdown
Contributor Author

The cause of the second issue was that when creating a node, editor always connects to the last selected node. However, if no node was selected, it assigned an empty ID, which is why it returned undefined. In short, it was another overlooked case. Thanks again. I’ll be looking forward to ROSCon!

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