Skip to content

Add flag to enable/disable userdata from the XODR parser in malidrive.#147

Merged
Santoi merged 1 commit intomainfrom
slopez/deprecate-user-data-parsing
Sep 9, 2025
Merged

Add flag to enable/disable userdata from the XODR parser in malidrive.#147
Santoi merged 1 commit intomainfrom
slopez/deprecate-user-data-parsing

Conversation

@Santoi
Copy link
Contributor

@Santoi Santoi commented Sep 9, 2025

🎉 New feature

Relates to maliput/maliput_malidrive#345

Summary

Test it

Easy copy commands:

maliput_query --maliput_backend=malidrive --xodr_file_path=src/maliput_malidrive/resources/Figure8.xodr --use_userdata_traffic_direction=true -- GetDirectionUsage 0_0_4
# Should output Bidirectional
maliput_query --maliput_backend=malidrive --xodr_file_path=src/maliput_malidrive/resources/Figure8.xodr --use_userdata_traffic_direction=false -- GetDirectionUsage 0_0_4
# Should output AgainstS

Actual input-output:

santi@39d3ed64527c:~/maliput_ws$ maliput_query --maliput_backend=malidrive --xodr_file_path=src/maliput_malidrive/resources/Figure8.xodr --use_userdata_traffic_direction=true -- GetDirectionUsage 0_0_4 
[INFO] Loading road network using malidrive backend implementation...

Road network loaded in 0.793944 seconds.
[INFO] RoadNetwork loaded successfully.

              : Result: Rule (0_0_4_8): Bidirectional
Elapsed Query Time: 1.4876e-05 s
santi@39d3ed64527c:~/maliput_ws$ maliput_query --maliput_backend=malidrive --xodr_file_path=src/maliput_malidrive/resources/Figure8.xodr --use_userdata_traffic_direction=false -- GetDirectionUsage 0_0_4 
[INFO] Loading road network using malidrive backend implementation...

Road network loaded in 0.797239 seconds.
[INFO] RoadNetwork loaded successfully.

              : Result: Rule (0_0_4_8): AgainstS
Elapsed Query Time: 1.5156e-05 s

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)

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.

Signed-off-by: Santiago Lopez <santiago.lopez@ekumenlabs.com>
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

@github-project-automation github-project-automation bot moved this from 🆕 Inbox to 💪 Approved in Maliput - Core development Sep 9, 2025
@Santoi Santoi merged commit c838fb3 into main Sep 9, 2025
3 checks passed
@Santoi Santoi deleted the slopez/deprecate-user-data-parsing branch September 9, 2025 18:51
@github-project-automation github-project-automation bot moved this from 💪 Approved to ✅ Done in Maliput - Core development Sep 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.

2 participants