-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: Fix issues on control & perception and improvement on handling control message. #2
fix: Fix issues on control & perception and improvement on handling control message. #2
Conversation
simulator/carla_autoware/README.md
Outdated
@@ -127,7 +127,6 @@ The `carla_ros.py` sets up the CARLA world: | |||
## Tips | |||
|
|||
- Misalignment might occurs during initialization, pressing `init by gnss` button should fix it. | |||
- Changing the `fixed_delta_seconds` can increase the simulation tick (default 0.05 s), some sensors params in `objects.json` need to be adjusted when it is changed (example: LIDAR rotation frequency have to match the FPS). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After this commit, it seems we should fix the rotation_frequency
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this tip should not be removed from the README.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the comment!
I agree to your point. I won't remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for your contribution. I tested your changes and the control is now perfect.
I have concerns about the performance impact of increasing the channels of the lidar, but we can add some info in the tips
about performance in another PR.
simulator/carla_autoware/README.md
Outdated
@@ -127,7 +127,6 @@ The `carla_ros.py` sets up the CARLA world: | |||
## Tips | |||
|
|||
- Misalignment might occurs during initialization, pressing `init by gnss` button should fix it. | |||
- Changing the `fixed_delta_seconds` can increase the simulation tick (default 0.05 s), some sensors params in `objects.json` need to be adjusted when it is changed (example: LIDAR rotation frequency have to match the FPS). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this tip should not be removed from the README.
Description
Here's the video for better description:
Tests performed
I respected the environment setting and followed testing steps written here.
Effects on system behavior
This resolves issue on oscillation and pointcloud range during the co-simulation using Carla, as the video above.
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.
After all checkboxes are checked, anyone who has write access can merge the PR.