Bring back quotation marks

happypose_examples/config/camera_info.yaml

@@ -1,11 +1,11 @@
 image_width: 640
 image_height: 480
-camera_name: webcam
+camera_name: "webcam"
 camera_matrix:
   rows: 3
   cols: 3
   data: [1066.778, 0.0, 312.9869, 0.0, 1067.487, 241.3109, 0.0, 0.0, 1.0]
-distortion_model: plumb_bob
+distortion_model: "plumb_bob"
 distortion_coefficients:
   rows: 1
   cols: 5

happypose_examples/config/cosypose_params.yaml

@@ -7,14 +7,14 @@
     # Extended verbosity on info logs. Show logs with number of detections and more
     verbose_info_logs: true
     # Specifies which pose estimator to use in the pipeline
-    pose_estimator_type: cosypose
+    pose_estimator_type: "cosypose"
     cosypose:
       # Name of a dataset used during training
       dataset_name: $(var dataset_name)
       # Object renderer parameters
       renderer:
         # Specifies which renderer to use in the pipeline
-        renderer_type: panda3d
+        renderer_type: "panda3d"
         # Number of CPU cores to use during rendering
         n_workers: 8
         # Render objects on a GPU
@@ -36,7 +36,7 @@
           #   - "ycbv-obj_000001"
     # List of camera names to subscribe to. Those names are internal to happypose_ros node
     # and can be chosen arbitrarily as long as they are valid YAML keys to be used later
-    camera_names: [cam_1]
+    camera_names: ["cam_1"]
     cameras:
       # Timeout, after which a frame from a camera is considered too old. Value '0.0' disables timeout
       timeout: 0.0

happypose_examples/config/cosypose_params_multiview.yaml

@@ -11,16 +11,16 @@
     # Extended verbosity on info logs. Show logs with number of detections and more
     verbose_info_logs: true
     # Which image time stamp to use in final detection message
-    time_stamp_strategy: oldest
+    time_stamp_strategy: "oldest"
     # Specifies which pose estimator to use in the pipeline
-    pose_estimator_type: cosypose
+    pose_estimator_type: "cosypose"
     cosypose:
       # Name of a dataset used during training
       dataset_name: $(var dataset_name)
       # Object renderer parameters
       renderer:
         # Specifies which renderer to use in the pipeline
-        renderer_type: panda3d
+        renderer_type: "panda3d"
         # Number of CPU cores to use during rendering
         n_workers: 8
         # Render objects on a GPU
@@ -50,7 +50,7 @@
           ba_n_iter: 100
     # List of camera names to subscribe to. Those names are internal to happypose_ros node
     # and can be chosen arbitrarily as long as they are valid YAML keys to be used later
-    camera_names: [cam_1, cam_2, cam_3]
+    camera_names: ["cam_1", "cam_2", "cam_3"]
     cameras:
       # Timeout, after which a frame from a camera is considered too old. Value '0.0' disables timeout
       timeout: 0.0

happypose_ros/happypose_ros/happypose_ros_parameters.yaml

@@ -1,117 +1,117 @@
 happypose_ros:
   device:
     type: string
-    default_value: cpu
-    description: Device to which the models will be loaded. Supported options are 'cpu' and 'cuda:x' where 'x' is the GPU number.
+    default_value: "cpu"
+    description: "Device to which the models will be loaded. Supported options are 'cpu' and 'cuda:x' where 'x' is the GPU number."
     validation:
       custom_validators::torch_check_device:
     read_only: true
   verbose_info_logs:
     type: bool
     default_value: false
-    description: Extended verbosity on info logs. Show logs with number of detections and more.
+    description: "Extended verbosity on info logs. Show logs with number of detections and more."
   time_stamp_strategy:
     type: string
-    default_value: oldest
-    description: Which image time stamp to use in final detection message.
+    default_value: "oldest"
+    description: "Which image time stamp to use in final detection message."
     validation:
-      one_of<>: [[average, newest, oldest]]
+      one_of<>: [["average", "newest", "oldest"]]
   visualization:
     publish_markers:
       type: bool
       default_value: false
-      description: Publish detected objects as markers to visualize detections in RViz.
+      description: "Publish detected objects as markers to visualize detections in RViz."
       read_only: true
     markers:
       dynamic_opacity:
         type: bool
         default_value: false
-        description: Change opacity of published markers based on their prediction score.
+        description: "Change opacity of published markers based on their prediction score."
       lifetime:
         type: double
         default_value: 10.0
-        description: Lifetime of a published marker.
+        description: "Lifetime of a published marker."
         validation:
           gt<>: [0.0]
   pose_estimator_type:
     type: string
-    default_value: cosypose
-    description: Specifies which pose estimator to use in the pipeline.
+    default_value: "cosypose"
+    description: "Specifies which pose estimator to use in the pipeline."
     validation:
-      one_of<>: [[cosypose]]
+      one_of<>: [["cosypose"]]
     read_only: true
   cosypose:
     dataset_name:
       type: string
-      default_value: ''
-      description: Name of a dataset used during training.
+      default_value: ""
+      description: "Name of a dataset used during training."
       validation:
-        one_of<>: [[hope, tless, ycbv]]
+        one_of<>: [["hope", "tless", "ycbv"]]
       read_only: true
     renderer:
       renderer_type:
         type: string
-        default_value: panda3d
-        description: Specifies which renderer to use in the pipeline.
+        default_value: "panda3d"
+        description: "Specifies which renderer to use in the pipeline."
         validation:
-          one_of<>: [[panda3d, bullet]]
+          one_of<>: [["panda3d", "bullet"]]
         read_only: true
       n_workers:
         type: int
         default_value: 8
-        description: Number of CPU cores to use during rendering.
+        description: "Number of CPU cores to use during rendering."
         validation:
           gt_eq<>: 1
         read_only: true
       gpu_renderer:
         type: bool
         default_value: true
-        description: Render objects on a GPU.
+        description: "Render objects on a GPU."
         read_only: true
     inference:
       detector:
         detection_th:
           type: double
           default_value: 0.7
-          description: Detection threshold of an object used by detector.
+          description: "Detection threshold of an object used by detector."
           validation:
             bounds<>: [0.0, 1.0]
       pose_estimator:
         n_refiner_iterations:
           type: int
           default_value: 3
-          description: Number of iterations for the refiner.
+          description: "Number of iterations for the refiner."
           validation:
             gt_eq<>: [1]
         n_coarse_iterations:
           type: int
           default_value: 1
-          description: Number of iterations for the coarse estimate.
+          description: "Number of iterations for the coarse estimate."
           validation:
             gt_eq<>: [1]
       multiview:
         ransac_n_iter:
           type: int
           default_value: 2000
-          description: Number of ransac iterations when matching views.
+          description: "Number of ransac iterations when matching views."
         ransac_dist_threshold:
           type: double
           default_value: 0.2
-          description: Threshold (in metter) on the symmetric distance (Mean Symmetry-Aware Surface Distance) used consider a tentative match as an inlier during RANSAC iterations.
+          description: "Threshold (in metter) on the symmetric distance (Mean Symmetry-Aware Surface Distance) used consider a tentative match as an inlier during RANSAC iterations."
         ba_n_iter:
           type: int
           default_value: 100
-          description: Number of steps in the final bundle adjustment refinement.
+          description: "Number of steps in the final bundle adjustment refinement."
       labels_to_keep:
         type: string_array
-        description: Labels of detected objects to keep. If not specified, all objects are kept.
+        description: "Labels of detected objects to keep. If not specified, all objects are kept."
         # Walkaround to obtain "empty" string array
-        default_value: ['']
+        default_value: [""]
         validation:
           unique<>:
   camera_names:
     type: string_array
-    description: List of names of cameras to subscribe.
+    description: "List of names of cameras to subscribe."
     read_only: true
     validation:
       size_gt<>: [0]
@@ -120,36 +120,36 @@ happypose_ros:
     timeout:
       type: double
       default_value: 0.0
-      description: Timeout, after which a frame from a camera is considered too old. Value '0.0' disables timeout.
+      description: "Timeout, after which a frame from a camera is considered too old. Value '0.0' disables timeout."
       validation:
         gt_eq<>: [0.0]
     n_min_cameras:
       type: int
       default_value: 1
-      description: Minimum number of valid camera views to start pose estimation pipeline.
+      description: "Minimum number of valid camera views to start pose estimation pipeline."
       validation:
         gt_eq<>: [1]
       read_only: true
     __map_camera_names:
       compressed:
         type: bool
         default_value: false
-        description: Expect compressed image messages from given camera.
+        description: "Expect compressed image messages from given camera."
         read_only: true
       leading:
         type: bool
         default_value: false
-        description: Consider the camera to be leading. If a camera is leading, its frame_id is used as a reference. Only one camera can be leading, and it can't publish TF at the same time.
+        description: "Consider the camera to be leading. If a camera is leading, its frame_id is used as a reference. Only one camera can be leading, and it can't publish TF at the same time."
         read_only: true
       publish_tf:
         type: bool
         default_value: false
-        description: Publish TF of a given camera relative to the leading camera.
+        description: "Publish TF of a given camera relative to the leading camera."
         read_only: true
       k_matrix:
         type: double_array
         default_value: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
-        description: Camera intrinsic matrix. If values are valid intrinsic matrix, overwrites values from info ROS topic.
+        description: "Camera intrinsic matrix. If values are valid intrinsic matrix, overwrites values from info ROS topic."
         validation:
           fixed_size<>: [9]
           lower_element_bounds<>: [0.0]

happypose_ros/test/test_multi_view.yaml

@@ -1,13 +1,13 @@
 /**:
   ros__parameters:
     verbose_info_logs: true
-    pose_estimator_type: cosypose
+    pose_estimator_type: "cosypose"
     cosypose:
-      dataset_name: ycbv
+      dataset_name: "ycbv"
       # reduce memory footprint during tests by reducing number of workers
       renderer:
         n_workers: 1
-    camera_names: [cam_1, cam_2, cam_3]
+    camera_names: ["cam_1", "cam_2", "cam_3"]
     cameras:
       timeout: 0.0
       n_min_cameras: 3

happypose_ros/test/test_single_view.yaml

@@ -2,14 +2,14 @@
   ros__parameters:
     # device is set from launch file
     verbose_info_logs: true
-    pose_estimator_type: cosypose
+    pose_estimator_type: "cosypose"
     visualization.publish_markers: true
     cosypose:
-      dataset_name: ycbv
+      dataset_name: "ycbv"
       # reduce memory footprint during tests by reducing number of workers
       renderer:
         n_workers: 1
-    camera_names: [cam_1]
+    camera_names: ["cam_1"]
     cameras:
       timeout: 0.0
       n_min_cameras: 1