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

Prefix packages with autoware_ prefix and keep code in autoware namespace #4569

Open
esteve opened this issue Mar 27, 2024 · 4 comments
Open
Assignees
Labels
component:system System design and integration.

Comments

@esteve
Copy link
Contributor

esteve commented Mar 27, 2024

Checklist

  • I've read the contribution guidelines.
  • I've searched other issues and no duplicate issues were found.
  • I've agreed with the maintainers that I can plan this task.

Description

See https://github.com/orgs/autowarefoundation/discussions/4097

Purpose

Keep Autoware code clearly separate from other projects by adding appropriate namespacing and prefixes

Possible approaches

Add autoware namespace and autoware_ prefix for package names

Definition of done

All packages are prefixed and the code is in the autoware namespace

List of packages in reverse topological order:

  • tier4_planning_launch
  • static_centerline_generator
  • behavior_velocity_planner
  • behavior_velocity_walkway_module
  • behavior_velocity_run_out_module
  • behavior_path_external_request_lane_change_module
  • behavior_path_avoidance_by_lane_change_module
  • tier4_control_launch
  • behavior_velocity_virtual_traffic_light_module
  • behavior_velocity_traffic_light_module
  • behavior_velocity_template_module
  • behavior_velocity_stop_line_module
  • behavior_velocity_speed_bump_module
  • behavior_velocity_out_of_lane_module
  • behavior_velocity_occlusion_spot_module
  • behavior_velocity_no_stopping_area_module
  • behavior_velocity_no_drivable_lane_module
  • behavior_velocity_intersection_module
  • behavior_velocity_dynamic_obstacle_stop_module
  • behavior_velocity_detection_area_module
  • behavior_velocity_crosswalk_module
  • behavior_velocity_blind_spot_module
  • behavior_path_start_planner_module
  • behavior_path_side_shift_module
  • behavior_path_lane_change_module
  • behavior_path_goal_planner_module
  • behavior_path_dynamic_avoidance_module
  • behavior_path_avoidance_module
  • trajectory_follower_node
  • tier4_perception_launch
  • behavior_velocity_planner_common
  • behavior_path_sampling_planner_module
  • behavior_path_planner
  • traffic_light_fine_detector
  • tier4_simulator_launch
  • tier4_localization_launch
  • tier4_autoware_api_launch
  • scenario_selector
  • sample_sensor_kit_launch
  • pure_pursuit
  • planning_validator
  • pid_longitudinal_controller
  • path_smoother
  • path_sampler
  • obstacle_velocity_limiter
  • obstacle_stop_planner
  • obstacle_cruise_planner
  • obstacle_avoidance_planner
  • mpc_lateral_controller
  • motion_velocity_smoother
  • image_projection_based_fusion
  • freespace_planner
  • detection_by_tracker
  • behavior_path_planner_common
  • awsim_sensor_kit_launch
  • vehicle_cmd_gate
  • trajectory_follower_base
  • tracking_object_merger
  • tier4_planning_rviz_plugin
  • tensorrt_yolox
  • surround_obstacle_checker
  • steer_offset_estimator
  • simple_planning_simulator
  • radar_object_tracker
  • radar_object_clustering
  • predicted_path_checker
  • pose_initializer
  • planning_topic_converter
  • planning_test_utils
  • planning_evaluator
  • perception_online_evaluator
  • path_distance_calculator
  • operation_mode_transition_manager
  • object_merger
  • multi_object_tracker
  • mission_planner
  • map_based_prediction
  • lidar_centerpoint
  • lane_departure_checker
  • frenet_planner
  • external_cmd_converter
  • euclidean_cluster
  • detected_object_validation
  • default_ad_api
  • control_validator
  • control_performance_analysis
  • common_sensor_launch
  • cluster_merger
  • bezier_sampler
  • autonomous_emergency_braking
  • ar_tag_based_localizer
  • accel_brake_map_calibrator
  • tier4_system_launch
  • tier4_sensing_launch
  • tier4_perception_rviz_plugin
  • tier4_map_launch
  • tier4_localization_rviz_plugin
  • sampler_common
  • raw_vehicle_cmd_converter
  • pose_estimator_arbiter
  • pacmod_interface
  • occupancy_grid_map_outlier_filter
  • obstacle_collision_checker
  • object_recognition_utils
  • nebula_sensor_driver
  • ndt_scan_matcher
  • motion_utils
  • lidar_apollo_segmentation_tvm_nodes
  • landmark_manager
  • ground_segmentation
  • global_parameter_loader
  • freespace_planning_algorithms
  • elevation_map_loader
  • compare_map_segmentation
  • yabloc_pose_initializer
  • yabloc_particle_filter
  • yabloc_image_processing
  • vehicle_info_util
  • traffic_light_recognition_marker_publisher
  • traffic_light_occlusion_predictor
  • traffic_light_map_based_detector
  • tier4_vehicle_rviz_plugin
  • tier4_system_rviz_plugin
  • tier4_control_rviz_plugin
  • tier4_automatic_goal_rviz_plugin
  • system_monitor
  • system_error_monitor
  • simple_object_merger
  • shape_estimation
  • route_handler
  • radar_tracks_msgs_converter
  • radar_static_pointcloud_filter
  • radar_fusion_to_detected_object
  • radar_crossing_objects_noise_filter
  • probabilistic_occupancy_grid_map
  • pose_instability_detector
  • pointcloud_preprocessor
  • objects_of_interest_marker_interface
  • object_velocity_splitter
  • nebula_tests
  • nebula_examples
  • map_projection_loader
  • map_loader
  • localization_util
  • localization_evaluator
  • localization_error_monitor
  • lidar_centerpoint_tvm
  • lidar_apollo_segmentation_tvm
  • lidar_apollo_instance_segmentation
  • kinematic_evaluator
  • interpolation
  • imu_corrector
  • image_diagnostics
  • gyro_odometer
  • grid_map_utils
  • goal_distance_calculator
  • gnss_poser
  • geo_pose_projector
  • fault_injection
  • external_cmd_selector
  • ekf_localizer
  • dummy_perception_publisher
  • dummy_diag_publisher
  • detected_object_feature_remover
  • crosswalk_traffic_light_estimator
  • awapi_awiv_adapter
  • ad_api_adaptors
  • yabloc_common
  • vehicle_velocity_converter
  • traffic_light_visualization
  • traffic_light_utils
  • traffic_light_multi_camera_fusion
  • traffic_light_classifier
  • traffic_light_arbiter
  • tier4_traffic_light_rviz_plugin
  • tier4_state_rviz_plugin
  • tier4_camera_view_rviz_plugin
  • tier4_calibration_rviz_plugin
  • tier4_autoware_utils
  • tier4_auto_msgs_converter
  • tier4_adapi_rviz_plugin
  • shift_decider
  • nebula_ros
  • mrm_handler
  • map_tf_generator
  • map_height_fitter
  • lanelet2_map_preprocessor
  • lanelet2_extension_python
  • joy_controller
  • geography_utils
  • emergency_handler
  • eagleye_rt
  • eagleye_gnss_converter
  • eagleye_geo_pose_converter
  • costmap_generator
  • component_interface_specs
  • bytetrack
  • automatic_pose_initializer
  • twist2accel
  • tier4_vehicle_msgs
  • tier4_perception_msgs
  • tier4_api_msgs
  • tensorrt_classifier
  • rtc_manager_rviz_plugin
  • object_range_splitter
  • nebula_hw_interfaces
  • mrm_emergency_stop_operator
  • mrm_comfortable_stop_operator
  • mission_planner_rviz_plugin
  • lanelet2_extension
  • hazard_status_converter
  • fake_test_node
  • external_velocity_limit_selector
  • eagleye_navigation
  • eagleye_geo_pose_fusion
  • eagleye_fix2kml
  • diagnostic_graph_aggregator
  • component_state_monitor
  • component_interface_utils
  • component_interface_tools
  • common_awsim_sensor_launch
  • yabloc_monitor
  • velodyne_monitor
  • vehicle_door_simulator
  • tvm_utility
  • tree_structured_parzen_estimator
  • topic_state_monitor
  • time_utils
  • tier4_vehicle_launch
  • tier4_target_object_type_rviz_plugin
  • tier4_system_msgs
  • tier4_simulated_clock_rviz_plugin
  • tier4_screen_capture_rviz_plugin
  • tier4_planning_msgs
  • tier4_pcl_extensions
  • tier4_metrics_rviz_plugin
  • tier4_logging_level_configure_rviz_plugin
  • tier4_hmi_msgs
  • tier4_debug_rviz_plugin
  • tier4_datetime_rviz_plugin
  • tier4_control_msgs
  • tier4_api_utils
  • stop_filter
  • signal_processing
  • rtklib_bridge
  • rtc_replayer
  • rtc_interface
  • radar_tracks_noise_filter
  • radar_threshold_filter
  • radar_scan_to_pointcloud2
  • qp_interface
  • pose2twist
  • polar_grid
  • perception_utils
  • osqp_interface
  • nebula_decoders
  • livox_tag_filter
  • kalman_filter
  • image_transport_decompressor
  • glog_component
  • eagleye_msgs
  • duplicated_node_checker
  • dummy_infrastructure
  • diagnostic_converter
  • cuda_utils
  • boost_udp_driver
  • boost_serial_driver
  • bluetooth_monitor
  • bag_time_manager_rviz_plugin
  • ad_api_visualizers
  • vls_description
  • tier4_v2x_msgs
  • tier4_simulation_msgs
  • tier4_rtc_msgs
  • tier4_map_msgs
  • tier4_localization_msgs
  • tier4_external_api_msgs
  • tier4_debug_msgs
  • tamagawa_imu_driver
  • sample_vehicle_launch
  • sample_vehicle_description
  • sample_sensor_kit_description
  • rtklib_msgs
  • robosense_msgs
  • radar_description
  • pointcloud_to_laserscan
  • pandar_msgs
  • pandar_description
  • nebula_common
  • ndt_omp
  • mussp
  • llh_converter
  • livox_description
  • individual_params
  • imu_description
  • eagleye_tf
  • eagleye_coordinate
  • eagleye_can_velocity_converter
  • camera_description
  • boost_tcp_driver
  • boost_io_context
  • awsim_sensor_kit_description
  • velodyne_description
  • tensorrt_common
  • morai_msgs
  • heaphook
@xmfcx
Copy link
Contributor

xmfcx commented May 2, 2024

Following up from:

We should prefix the package folder names with autoware_ too.

The package name in the package.xml file should match its parent folder.

I might have missed something too, open to discussion.

@esteve
Copy link
Contributor Author

esteve commented May 2, 2024

@xmfcx I've been waiting for feedback from the TierIV engineers regarding renaming the folders. @mitsudome-r do you have any updates? Thanks.

@xmfcx
Copy link
Contributor

xmfcx commented May 2, 2024

I would like to change it and see the response in this case. And solve the issues after they arise.

main branch is a development branch and this is expected.

@mitsudome-r
Copy link
Member

I think we can move on with changing the package folder name as well.
However, we did have some concerns about the changing the package names & include folder structure for the packages in autoware_common repository and also the packages with tier4_ prefix.

https://github.com/autowarefoundation/autoware_common

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:system System design and integration.
Projects
Status: In Progress
Development

No branches or pull requests

4 participants