Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Completed and fixed the reg elem visualization #168

Merged
merged 5 commits into from
Jan 8, 2025

Conversation

JStephenHuang
Copy link

Fixed the reg elem visualization by using the custom reg elem classes and made mock function to get TrafficLightState enum.

@JStephenHuang JStephenHuang changed the title Jshuang/hd map/viz Completed and fixed the reg elem visualization Jan 5, 2025
@JStephenHuang
Copy link
Author

JStephenHuang commented Jan 5, 2025

When testing the traffic light element viz, dont forget to change the topic name to "/carla/traffic_lights/detection3d_array" in the hd_map_service "traffic_light" subscriber.

Copy link
Contributor

@VishGit1234 VishGit1234 left a comment

Choose a reason for hiding this comment

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

Great Work, was able to run it on my end and see the visualized traffic lights.
The hardcoded topic names have to be changed to ros parameters but that can be done in the hd-map branch directly.

@VishGit1234 VishGit1234 merged commit 422dda6 into world-modeling-hd-map Jan 8, 2025
@VishGit1234 VishGit1234 deleted the jshuang/hd-map/viz branch January 8, 2025 05:57
JStephenHuang added a commit that referenced this pull request Jan 21, 2025
* Fixed traffic visualization

* fix: viz reg elem using custom class

* member declarations should match member initialization list

* undo notebook changes

---------

Co-authored-by: JStephenHuang <[email protected]>
Co-authored-by: Vishal Jayakumar <[email protected]>
Co-authored-by: VishGit1234 <[email protected]>
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