Skip to content

remove the cyan and magenta circles which show lidar mounts#541

Merged
griswaldbrooks merged 1 commit intomainfrom
remove-lidar-circles
Mar 25, 2026
Merged

remove the cyan and magenta circles which show lidar mounts#541
griswaldbrooks merged 1 commit intomainfrom
remove-lidar-circles

Conversation

@bkanator
Copy link

@bkanator bkanator commented Mar 24, 2026

verified cyan and magenta circles removed

image

@bkanator bkanator self-assigned this Mar 24, 2026
@bkanator bkanator added this to the 9.1.0 milestone Mar 24, 2026
@bkanator bkanator requested a review from griswaldbrooks March 24, 2026 18:30
Copy link

@griswaldbrooks griswaldbrooks left a comment

Choose a reason for hiding this comment

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

Please retarget to v9.1 branch

@bkanator bkanator changed the base branch from main to v9.1 March 24, 2026 22:37
@bkanator
Copy link
Author

bkanator commented Mar 24, 2026

fyi original lidar work went into main not v9.1 for the example_ws. So the question is do I bring all the lidar work plus disk removal into v9.1 in example_ws? and main? looks like a kortex PR would also need to be part of the main to v9.1 migration which is also not a part of this...

I'm keeping it targeted to main for now since that is where the lidar changes live in this repo

@bkanator bkanator changed the base branch from v9.1 to main March 24, 2026 22:46
@bkanator bkanator force-pushed the remove-lidar-circles branch from fb3db68 to 0a81cad Compare March 24, 2026 22:46
@griswaldbrooks griswaldbrooks merged commit cdd80d0 into main Mar 25, 2026
10 checks passed
@griswaldbrooks griswaldbrooks deleted the remove-lidar-circles branch March 25, 2026 14:29
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