-
Notifications
You must be signed in to change notification settings - Fork 444
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
Start preparing carla native ros2 interface #728
Open
berndgassmann
wants to merge
34
commits into
carla-simulator:master
Choose a base branch
from
berndgassmann:berndgassmann/carla_native_ros2_interface
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Start preparing carla native ros2 interface #728
berndgassmann
wants to merge
34
commits into
carla-simulator:master
from
berndgassmann:berndgassmann/carla_native_ros2_interface
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Wait one tick before creating carla actors to ensure the transforms of the newly spawned actors have been updated by the server
and fix task_queue assignment
object_type attribute is empty, therefore guess the object class based on the blueprint name as a workaround
Fill the message header of CarlaEgoVehicleControl messages with header received from odometry Some robustness increases: - make vehicle_pid_controller robust against missing speed/pose input
Don't execute planner step if no odometry message was received
The number of vegetation bouunding boxes can explode due to the duplication of small vegetation pieces. Millions of bounding boxes are not useful, taking minutes to load and consuming gigabytes of memory.
… into berndgassmann/dev
…o berndgassmann/dev
This allows for using multiple carla_ad_agents within a scenario.
berndgassmann
force-pushed
the
berndgassmann/carla_native_ros2_interface
branch
2 times, most recently
from
July 16, 2024 16:49
cb1df36
to
c97bb79
Compare
berndgassmann
force-pushed
the
berndgassmann/carla_native_ros2_interface
branch
from
July 22, 2024 12:18
c97bb79
to
c821621
Compare
Since Carla native interface directly accepts ackermann commands, this is not required anymore.
To replace /carla/markers published by the old ros bridge.
- make local_planner subscription to CarlaVehicleStatus best effort and extend ros compatibility qos setting - adapt cala_walker_agent to carla native ros interface - fix derived_objects_visualizer
This provides the pseudo.speedometer and pseudo.odom sensors for better backward compatibility and usability.
allowing to select the vehicle to be manually controlled
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Should be merged into new branch of ROS brigde carla_native_ros2_interface where the work on the native interface will have to be conducted.
This change is