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

style: update autoware user interface to the latest ui #1349

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KhalilSelyan
Copy link

Description

Updated the .rviz config file to include the correct layout

Related issue

@brkay54
Copy link

brkay54 commented Aug 22, 2024

cc @hakuturu583

Thank you for your work. I can say that the configuration appears to be the same as PSim's configuration.

Screenshot from 2024-08-22 14-21-18

I can suggest the following improvements:

  • The Condition Group marker may locate and scale better. (it is not related this improvement, but words are too big to read.)

image

  • With the default configuration, the trajectory can not be seen. The simulation already publishes the lanelet map so we can disable LaneletMap in the Map folder.

image

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
@KhalilSelyan KhalilSelyan force-pushed the Update-Autoware-User-Interface-to-the-latest-UI branch from 731db3e to b98b2ce Compare August 23, 2024 09:18
@hakuturu583 hakuturu583 self-requested a review August 27, 2024 02:19
@hakuturu583
Copy link
Collaborator

Thank you for sending Pull request!
But...

The rviz plugin added in this PR has its source code within Autoware, and to incorporate this PR, it would be necessary to add Autoware as a build dependency.
However, scenario_simulator_v2 is intentionally structured to depend only on the released packages of Autoware in order to minimize its dependency on Autoware.

@hakuturu583
Copy link
Collaborator

hakuturu583 commented Aug 27, 2024

We are carrying out the release process in this Issue.
We plan to contribute there, release the rviz plugin, and then incorporate this PR.
I apologize for the time this is taking.

@KhalilSelyan KhalilSelyan marked this pull request as draft September 24, 2024 07:15
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.

3 participants