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

feat: enable side lidars and adjust positions #54

Merged

Conversation

mozhoku
Copy link
Collaborator

@mozhoku mozhoku commented Apr 18, 2024

Description

Enabled side lidars on the Lexus by default and adjusted positions slightly

Related links

Related Issue:

Related PR:

Tests performed

I've tested autonomous driving in Nishishinjuku environment with the side/top lidars enabled with the new configuration.

Effects on system behavior

No effect.

Notes for reviewers

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mozhoku mozhoku self-assigned this Apr 18, 2024
@mozhoku mozhoku added type:bug Software flaws or errors. type:new-feature New functionalities or additions, feature requests. labels Apr 18, 2024
@mozhoku mozhoku requested review from brkay54 and xmfcx April 18, 2024 09:15
@mozhoku mozhoku marked this pull request as ready for review April 18, 2024 09:26
Copy link
Member

@brkay54 brkay54 left a comment

Choose a reason for hiding this comment

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

I run this configuration with cutting-edge Autoware and the results are seems good to me! Thank you for the work!

Raw Pointcloud for the side lidars:
fixed2
fixed1

@xmfcx
Copy link

xmfcx commented Apr 24, 2024

could you rebase this to the main?

@mozhoku mozhoku force-pushed the chore/adjust-lidar-positions-for-lexus branch from 073f59c to 29aa85f Compare April 24, 2024 14:48
Signed-off-by: Alptuğ Cırıt <[email protected]>
@mozhoku mozhoku force-pushed the chore/adjust-lidar-positions-for-lexus branch from 29aa85f to ea8b572 Compare April 24, 2024 14:56
@xmfcx xmfcx changed the title chore: enable side lidars and adjust positions feat: enable side lidars and adjust positions Apr 24, 2024
@xmfcx xmfcx closed this Apr 24, 2024
@xmfcx xmfcx reopened this Apr 24, 2024
@xmfcx xmfcx merged commit 78d6387 into autowarefoundation:main Apr 24, 2024
6 checks passed
@mozhoku mozhoku deleted the chore/adjust-lidar-positions-for-lexus branch May 2, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Software flaws or errors. type:new-feature New functionalities or additions, feature requests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants