Skip to content

Provides an integration application to export a RoadGeometry as a DOT file#139

Merged
francocipollone merged 3 commits intomainfrom
agalbachicar/#543_create_dot_file_from_routing_graph
Jun 9, 2025
Merged

Provides an integration application to export a RoadGeometry as a DOT file#139
francocipollone merged 3 commits intomainfrom
agalbachicar/#543_create_dot_file_from_routing_graph

Conversation

@agalbachicar
Copy link
Collaborator

🎉 New feature

Pairs with maliput/maliput#637

Summary

Try it by calling:

$ maliput_to_dot --maliput_backend malidrive --xodr_file_path ./src/maliput_malidrive/resources/TShapeRoad.xodr

And then see the output value:

$ cat maliput_graph.dot
graph {
1 -- 5 [ label = "9_0" ];
5 -- 1 [ label = "8_0" ];
5 -- 2 [ label = "7_0" ];
1 -- 2 [ label = "5_0" ];
4 -- 5 [ label = "2_0" ];
2 -- 5 [ label = "6_0" ];
1 -- 2 [ label = "4_0" ];
2 -- 3 [ label = "1_0" ];
0 -- 1 [ label = "0_0" ];
}

Rendering can be seen here.

Test it

See instructions above.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if it affects the public API)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

… file.

Try it by calling:

$ maliput_to_dot --maliput_backend malidrive --xodr_file_path ./src/maliput_malidrive/resources/TShapeRoad.xodr

And then see the output value:

$ cat maliput_graph.dot
graph {
1 -- 5 [ label = "9_0" ];
5 -- 1 [ label = "8_0" ];
5 -- 2 [ label = "7_0" ];
1 -- 2 [ label = "5_0" ];
4 -- 5 [ label = "2_0" ];
2 -- 5 [ label = "6_0" ];
1 -- 2 [ label = "4_0" ];
2 -- 3 [ label = "1_0" ];
0 -- 1 [ label = "0_0" ];
}

Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
francocipollone
francocipollone previously approved these changes Jul 5, 2024
Copy link
Collaborator

@liangfok liangfok left a comment

Choose a reason for hiding this comment

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

Completed a pass.

Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
Copy link
Collaborator

@liangfok liangfok left a comment

Choose a reason for hiding this comment

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

LGTM with minor comment.


/// @file maliput_to_dot.cc
///
/// Builds a dragway, multilane or malidrive road geometry
Copy link
Collaborator

Choose a reason for hiding this comment

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

This could be replaced with "See kUsageMessage below for details." That'll avoid the need to keep them consistent.

Copy link
Collaborator

@francocipollone francocipollone left a comment

Choose a reason for hiding this comment

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

LGTM!
Do you need to retrigger CI?

@francocipollone francocipollone merged commit 1bfe5dd into main Jun 9, 2025
3 checks passed
@francocipollone francocipollone deleted the agalbachicar/#543_create_dot_file_from_routing_graph branch June 9, 2025 17:00
@github-project-automation github-project-automation bot moved this from 💪 Approved to ✅ Done in Maliput - Core development Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants