-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Feature/gazebo simulator #1930
Feature/gazebo simulator #1930
Conversation
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.
Great job.
Do you want to perfectly deprecate system/gazebo/*
, right? Please describe deprecation and the overview of your simulator to #1907.
ros/src/simulation/gazebo_simulator/launcher/vehicle_gazebo_simulation_launcher/package.xml
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/launcher/vehicle_gazebo_simulation_launcher/CMakeLists.txt
Show resolved
Hide resolved
...ation/gazebo_simulator/launcher/vehicle_gazebo_simulation_launcher/scripts/download_model.sh
Outdated
Show resolved
Hide resolved
...imulator/vehicle/vehicle_gazebo_simulation_interface/src/vehicle_gazebo_input_subscriber.cpp
Outdated
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/vehicle/vehicle_model/urdf/vehicle.xacro
Outdated
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/vehicle/vehicle_model/urdf/wheel.gazebo
Outdated
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/vehicle/vehicle_model/package.xml
Outdated
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/worlds/gazebo_world_description/package.xml
Outdated
Show resolved
Hide resolved
…eature/gazebo_simulator
…ption/package.xml Co-Authored-By: yukkysaito <yukky.saito@gmail.com>
…age.xml Co-Authored-By: yukkysaito <yukky.saito@gmail.com>
…eature/gazebo_simulator
Thank you. fixed bug |
Fix/gazebo_simulator CI
@yukkysaito the velodyne_simulator metapackage should not be added as a dependency. #1985 |
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.
Better than before, good job.
It is possible to fix details even after merging, so these are just my comments.
ros/src/simulation/gazebo_simulator/launcher/vehicle_gazebo_simulation_launcher/CMakeLists.txt
Show resolved
Hide resolved
..._simulator/vehicle/vehicle_gazebo_simulation_interface/src/vehicle_gazebo_info_publisher.cpp
Show resolved
Hide resolved
...imulator/vehicle/vehicle_gazebo_simulation_interface/src/vehicle_gazebo_input_subscriber.cpp
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/vehicle/vehicle_model/CMakeLists.txt
Show resolved
Hide resolved
ros/src/simulation/gazebo_simulator/vehicle/vehicle_gazebo_simulation_interface/CMakeLists.txt
Show resolved
Hide resolved
…eature/gazebo_simulator
@yukkysaito Great contribution! In the near future, let's integrate real/sim descriptions and create the general package. |
@yukkysaito is there any difference between your/CPFL car_demo fork and the upstream version? |
…imulator Feature/gazebo simulator
Status
PRODUCTION / DEVELOPMENT
Description
https://github.com/CPFL/Autoware/issues/1907
In the initial release, it runs with runtime manager, but ultimately the UI is maintained with a new autoware launcher
https://github.com/CPFL/Autoware/tree/feature/autoware_launcher_py
Todos