Skip to content

Visualize brake and indicator lights #115

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

Conversation

buraktiryaki
Copy link
Contributor

@buraktiryaki buraktiryaki commented Aug 1, 2025

@buraktiryaki buraktiryaki force-pushed the feat/visualize-brake-and-indicator-lights branch from 2153d71 to 2f6c6d1 Compare August 1, 2025 12:51
@buraktiryaki buraktiryaki force-pushed the feat/visualize-brake-and-indicator-lights branch from 2f6c6d1 to f3fbf85 Compare August 1, 2025 13:35
@jdsika
Copy link
Contributor

jdsika commented Aug 4, 2025

Please add a test .mcap file to showcase the feature

@jdsika jdsika added the enhancement New feature or request label Aug 5, 2025
@jdsika jdsika added this to the v0.0.7 milestone Aug 5, 2025
@rmessaou rmessaou merged commit 4dd0b57 into lichtblick-suite:main Aug 5, 2025
1 check passed
Copy link
Contributor

@jdsika jdsika left a comment

Choose a reason for hiding this comment

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

  • Cubes should not be outside of the vehicle bounding box
  • Placement shall work for differently sized vehicles
  • Please add proper exception handling for corrupt/missing data input
  • Are placement constants derived from specifications/norms?

@jdsika
Copy link
Contributor

jdsika commented Aug 5, 2025

Also check the behavior with motorcycles and scooters etc which can also be meant if it has a "vehicleClassification"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants