Merge pull request #4 from nobleo/sort

Inline comments to allow for sorting

Author: Tobias-Fischer

vinca.yaml

@@ -73,8 +73,7 @@ packages_select_by_deps:
   - moveit-servo
   - moveit-visual-tools
   - moveit-ros-trajectory-cache
-  # Requested in https://www.linkedin.com/feed/update/urn:li:activity:7346559234177703938/
-  - moveit-chomp-optimizer-adapter
+  - moveit-chomp-optimizer-adapter  # Requested in https://www.linkedin.com/feed/update/urn:li:activity:7346559234177703938/
   - moveit-ros-control-interface
 
   - ros2_control
@@ -95,11 +94,9 @@ packages_select_by_deps:
   - ackermann-msgs
   - sbg_driver
 
-  # requested in https://github.com/RoboStack/ros-humble/issues/249
-  - twist_mux
+  - twist_mux  # requested in https://github.com/RoboStack/ros-humble/issues/249
 
-  # requested in https://github.com/RoboStack/ros-humble/issues/252
-  - rmw_zenoh_cpp
+  - rmw_zenoh_cpp  # requested in https://github.com/RoboStack/ros-humble/issues/252
 
   - flex_sync
   - gripper_controllers
@@ -153,8 +150,7 @@ packages_select_by_deps:
   - behaviortree_cpp
 
   - gtsam
-  # Requested in https://github.com/RoboStack/ros-humble/issues/325
-  - data_tamer_cpp
+  - data_tamer_cpp  # Requested in https://github.com/RoboStack/ros-humble/issues/325
 
   - proxsuite
 
@@ -184,11 +180,9 @@ packages_select_by_deps:
   # These packages are only built on Linux as they depend on Linux-specific API
   - if: linux
     then:
-      # Depends on socketcan
-      - nobleo_socketcan_bridge
-      - ros2_socketcan
-      # Depends on v4l
-      - usb_cam
+      - nobleo_socketcan_bridge  # Depends on socketcan
+      - ros2_socketcan  # Depends on socketcan
+      - usb_cam  # Depends on v4l
 
       - libcamera
 
@@ -200,14 +194,10 @@ packages_select_by_deps:
     then:
       - py_binding_tools
       - livox_ros_driver2
-      # Depends on libcamera that is only available on linux
-      - camera_ros
-      # Depends on v4l that is only available on linux
-      - v4l2_camera
-      # Serial communication only implemented for linux
-      - swri_serial_util
-      # Depends on camera_ros
-      - apriltag_ros
+      - camera_ros  # Depends on libcamera that is only available on linux
+      - v4l2_camera  # Depends on v4l that is only available on linux
+      - swri_serial_util  # Serial communication only implemented for linux
+      - apriltag_ros  # Depends on camera_ros
 
   # These packages are currently not build on Windows, but they be with some work
   - if: not win
@@ -223,8 +213,7 @@ packages_select_by_deps:
       - pinocchio
       - serial_driver
       - swri_console  # Until sync of: https://github.com/ros/rosdistro/pull/49750
-      # TODO on windows: fix iconv link issue
-      - ffmpeg_image_transport
+      - ffmpeg_image_transport  # TODO on windows: fix iconv link issue
       - apriltag_draw
       - apriltag_tools
       - apriltag_detector_umich
@@ -235,29 +224,19 @@ packages_select_by_deps:
       - rqt_tf_tree
       - rviz_satellite
       - odom_to_tf_ros2
-      # TODO on windows: cannot open pcl_io.lib
-      - ouster_ros
-      # Windows error: error C3861: '__builtin_unreachable': identifier not found
-      - moveit-hybrid-planning
-      # Windows error: error C2765: 'function' : an explicit specialization of a function template cannot have any default arguments
-      - autoware_motion_utils
-      # depends on autoware_motion_utils
-      - autoware_trajectory
-      - autoware_pose_initializer
-      - autoware_core_control
-      # Windows error: error C2338: static_assert failed: 'First argument to logging macros must be an rclcpp::Logger'
-      - autoware_ekf_localizer
-      # depends on autoware_ekf_localizer
-      - autoware_core_localization
-      # Windows error: error C3546: '...': there are no parameter packs available to expand
-      # Windows error: error C2678: binary '|': no operator found which takes a left-hand operand of type 'ranges::transform_view<ranges::ref_view<const T>,Arg>'
-      - autoware_lanlet2_utils
+      - ouster_ros  # TODO on windows: cannot open pcl_io.lib
+      - moveit-hybrid-planning  # Windows error: error C3861: '__builtin_unreachable': identifier not found
+      - autoware_motion_utils  # Windows error: error C2765: 'function': an explicit specialization of a function template cannot have any default arguments
+      - autoware_trajectory  # depends on autoware_motion_utils
+      - autoware_pose_initializer  # depends on autoware_motion_utils
+      - autoware_core_control  # depends on autoware_motion_utils
+      - autoware_ekf_localizer  # Windows error: error C2338: static_assert failed: 'First argument to logging macros must be an rclcpp::Logger'
+      - autoware_core_localization  # depends on autoware_ekf_localizer
+      - autoware_lanlet2_utils  # Windows errors: C3546 (no parameter packs to expand), C2678 (no operator '|' for transform_view)
       - autoware_osqp_interface
       - autoware_qp_interface
       - autoware_core
-      # most of the dep of grid_map work on Windows, but there are some rviz linking problems in octomap_rviz_plugins,
-      # see https://github.com/RoboStack/ros-jazzy/pull/79#issuecomment-2993499990
-      - grid_map
+      - grid_map  # rviz linking problems on Windows, see https://github.com/RoboStack/ros-jazzy/pull/79#issuecomment-2993499990
       - rslidar_sdk
       # mocap4r2 packages
       - mocap4r2_control