-
Notifications
You must be signed in to change notification settings - Fork 438
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
Editing pcl_recorder CMakeLists.txt for successful catkin build. #635
Open
chngdickson
wants to merge
711
commits into
0.9.6
Choose a base branch
from
master
base: 0.9.6
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.
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
…e accordingly, corrected list of sensors in config file, correct some bugs in carla_spawn_actors
…actors in ros_bridge tests and dockerfile
This reverts commit 3b5881d.
This reverts commit c999e31.
Spawn actors leaderboard
* using relative tfs for sensors in sync mode * merged leaderboard branch * using relative spawn pose only in sensors * uncomment tf publication
* added configure_world parameter * Merge branch 'leaderboard' into joel-mb/ticking * added passive parameter * added warning passive mode and sync mode
* using services to spawn spectator camera
* removed waiting for world info topic
This reverts commit 427ad4f.
Integrate leaderboard changes
Fix actor_factory and ego_vehicle
* avoid different ways to import carla_ad_agent submodules * ad agent refactoring * fix initial type objects variable * Merge branch 'master' into joel-mb/route_service * traffic light sensor using latched topics again * ad agent docs updated
Co-authored-by: joel-mb <[email protected]>
…enario Carla_ad_agent: Use target_speed from scenario
This PR refactors the ros_compatibility backend. The new backend tries to copy as much as possible the ROS2 API. All the other packages have been updated consequently.
Gets bounding boxes for some static objects from Carla, and converts them into Markers to be displayed in rviz. Co-authored-by: fpasch <[email protected]> Co-authored-by: Pasch, Frederik <[email protected]> Co-authored-by: Joel Moriana <[email protected]>
When generating the header field for some sensors, the timestamp now is used. This is leading to race conditions and synchronization issues between some sensors. This PR fixes the timestamp by using the timestamp provided by the sensor data.
…552) Fixes a bug when spawning a vehicle with rolename other than ego_vehicle.
Adds a new parameter (register_all_sensors) to avoid publishing data of all the sensors present in the simulation. When enabled, only sensors spawned by the bridge are registered. Otherwise, all the sensors present in the simulation are registered (current behavior).
Co-authored-by: joel-mb <[email protected]>
Co-authored-by: joel-mb <[email protected]>
* Change trigger text for running scenario * Added 0.9.11 section to changelog Co-authored-by: Pasch, Frederik <[email protected]> Co-authored-by: Joel Moriana <[email protected]>
) Co-authored-by: Stefan Roos <[email protected]>
carla_ackermann_control: - fill header of output messages - use simulated time to check for last received ackerman msg, otherwhise no contol command is published anymore if frames in sync mode last for too long Co-authored-by: joel-mb <[email protected]>
…569) Co-authored-by: Baltashov Ilia <[email protected]> Co-authored-by: joel-mb <[email protected]>
Fixed lidar topic path Co-authored-by: joel-mb <[email protected]>
* Fix cmake check format * Increased client timeout to 10s
* Fix CI rosversion cmd missing * Add missing rospy dependency * rospy dependency actually not missing fixing workflow ordering * Add rospy package again * disabling melodic temporarily Co-authored-by: Joel Moriana <[email protected]>
* Added missing dependencies * Fixed cmake format
* Fix scenario runner node for foxy * CHANGELOG updated
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.
There was a slight issue while compiling this package shown in the picture below.
Upon adding this line to the CMakeLists, i was able to build/compile it successfully.
add_definitions(-DBOOST_ERROR_CODE_HEADER_ONLY
CMakeLists.txt
This change is