fix(perception): fix perception docs (autowarefoundation#9766)

vividf · technolojin · pre-commit-ci[bot] · kminoda · commit 358afadf994c · 2024-12-27T11:24:12.000+09:00
* fix: fix perception docs

Signed-off-by: vividf &lt;yihsiang.fang@tier4.jp&gt;

* fix: fix missing parameter in schema

Signed-off-by: vividf &lt;yihsiang.fang@tier4.jp&gt;

* Update perception/autoware_object_merger/schema/data_association_matrix.schema.json

Co-authored-by: Taekjin LEE &lt;technolojin@gmail.com&gt;

* Update perception/autoware_object_merger/schema/data_association_matrix.schema.json

Co-authored-by: Taekjin LEE &lt;technolojin@gmail.com&gt;

* Update perception/autoware_object_merger/schema/data_association_matrix.schema.json

Co-authored-by: Taekjin LEE &lt;technolojin@gmail.com&gt;

* Update perception/autoware_object_merger/schema/data_association_matrix.schema.json

Co-authored-by: Taekjin LEE &lt;technolojin@gmail.com&gt;

* style(pre-commit): autofix

* chore: seperate paramters for different nodes

Signed-off-by: vividf &lt;yihsiang.fang@tier4.jp&gt;

---------

Signed-off-by: vividf &lt;yihsiang.fang@tier4.jp&gt;
Co-authored-by: Taekjin LEE &lt;technolojin@gmail.com&gt;
Co-authored-by: pre-commit-ci[bot] &lt;66853113+pre-commit-ci[bot]@users.noreply.github.com&gt;
diff --git a/perception/autoware_object_merger/README.md b/perception/autoware_object_merger/README.md
@@ -25,8 +25,16 @@ The successive shortest path algorithm is used to solve the data association pro
 
 ## Parameters
 
+- object association merger
+
 {{ json_to_markdown("perception/autoware_object_merger/schema/object_association_merger.schema.json") }}
+
+- data association matrix
+
 {{ json_to_markdown("perception/autoware_object_merger/schema/data_association_matrix.schema.json") }}
+
+- overlapped judge
+
 {{ json_to_markdown("perception/autoware_object_merger/schema/overlapped_judge.schema.json") }}
 
 ## Tips
diff --git a/perception/autoware_object_merger/schema/data_association_matrix.schema.json b/perception/autoware_object_merger/schema/data_association_matrix.schema.json
@@ -2,40 +2,81 @@
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "Data Association Matrix Parameters",
   "type": "object",
-  "properties": {
-    "ros__parameters": {
+  "definitions": {
+    "data_association_matrix": {
       "type": "object",
       "properties": {
         "can_assign_matrix": {
           "type": "array",
           "items": {
             "type": "number"
           },
-          "description": "Assignment table for data association"
+          "description": "Assignment table for data association.",
+          "default": [
+            0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 1,
+            0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0, 0, 0, 0, 0, 1, 1, 1, 0, 0,
+            0, 0, 0, 1, 1, 1
+          ]
         },
         "max_dist_matrix": {
           "type": "array",
           "items": {
             "type": "number"
           },
-          "description": "Maximum distance table for data association"
+          "description": "Maximum distance table for data association.",
+          "default": [
+            4.0, 4.0, 5.0, 5.0, 5.0, 2.0, 2.0, 2.0, 4.0, 2.0, 5.0, 5.0, 5.0, 1.0, 1.0, 1.0, 5.0,
+            5.0, 5.0, 5.0, 5.0, 1.0, 1.0, 1.0, 5.0, 5.0, 5.0, 5.0, 5.0, 1.0, 1.0, 1.0, 5.0, 5.0,
+            5.0, 5.0, 5.0, 1.0, 1.0, 1.0, 2.0, 1.0, 1.0, 1.0, 1.0, 3.0, 3.0, 3.0, 2.0, 1.0, 1.0,
+            1.0, 1.0, 3.0, 3.0, 3.0, 2.0, 1.0, 1.0, 1.0, 1.0, 3.0, 3.0, 2.0
+          ]
         },
         "max_rad_matrix": {
           "type": "array",
           "items": {
-            "type": "number"
+            "type": "number",
+            "minimum": 0.0
           },
-          "description": "Maximum angle table for data association. If value is greater than pi, it will be ignored."
+          "description": "Maximum angle table for data association. If value is greater than pi, it will be ignored.",
+          "default": [
+            3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 1.047, 1.047, 1.047, 1.047, 3.15,
+            3.15, 3.15, 3.15, 1.047, 1.047, 1.047, 1.047, 3.15, 3.15, 3.15, 3.15, 1.047, 1.047,
+            1.047, 1.047, 3.15, 3.15, 3.15, 3.15, 1.047, 1.047, 1.047, 1.047, 3.15, 3.15, 3.15,
+            3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15,
+            3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15, 3.15
+          ]
         },
         "min_iou_matrix": {
           "type": "array",
           "items": {
-            "type": "number"
+            "type": "number",
+            "minimum": 0.0
           },
-          "description": "Minimum IoU threshold matrix for data association. If value is negative, it will be ignored."
+          "description": "Minimum IoU threshold matrix for data association. If value is negative, it will be ignored.",
+          "default": [
+            0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.3, 0.2, 0.2, 0.2, 0.1, 0.1, 0.1, 0.1,
+            0.2, 0.3, 0.3, 0.3, 0.1, 0.1, 0.1, 0.1, 0.2, 0.3, 0.3, 0.3, 0.1, 0.1, 0.1, 0.1, 0.2,
+            0.3, 0.3, 0.3, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1,
+            0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1, 0.1
+          ]
+        }
+      },
+      "required": ["can_assign_matrix", "max_dist_matrix", "max_rad_matrix", "min_iou_matrix"],
+      "additionalProperties": false
+    }
+  },
+  "properties": {
+    "/**": {
+      "type": "object",
+      "properties": {
+        "ros__parameters": {
+          "$ref": "#/definitions/data_association_matrix"
         }
       },
-      "required": ["can_assign_matrix", "max_dist_matrix", "max_rad_matrix", "min_iou_matrix"]
+      "required": ["ros__parameters"],
+      "additionalProperties": false
     }
-  }
+  },
+  "required": ["/**"],
+  "additionalProperties": false
 }
diff --git a/perception/autoware_object_merger/schema/object_association_merger.schema.json b/perception/autoware_object_merger/schema/object_association_merger.schema.json
@@ -45,7 +45,8 @@
         "remove_overlapped_unknown_objects",
         "base_link_frame_id",
         "priority_mode"
-      ]
+      ],
+      "additionalProperties": false
     }
   },
   "properties": {
@@ -56,8 +57,10 @@
           "$ref": "#/definitions/object_association_merger"
         }
       },
-      "required": ["ros__parameters"]
+      "required": ["ros__parameters"],
+      "additionalProperties": false
     }
   },
-  "required": ["/**"]
+  "required": ["/**"],
+  "additionalProperties": false
 }
diff --git a/perception/autoware_object_merger/schema/overlapped_judge.schema.json b/perception/autoware_object_merger/schema/overlapped_judge.schema.json
@@ -2,26 +2,45 @@
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "Overlapped Judge Parameters",
   "type": "object",
-  "properties": {
-    "ros__parameters": {
+  "definitions": {
+    "overlapped_judge": {
       "type": "object",
       "properties": {
         "distance_threshold_list": {
           "type": "array",
           "items": {
             "type": "number"
           },
-          "description": "Distance threshold for each class used in judging overlap."
+          "description": "Distance threshold for each class used in judging overlap.",
+          "default": [9.0, 9.0, 9.0, 9.0, 9.0, 9.0, 9.0, 9.0]
         },
         "generalized_iou_threshold": {
           "type": "array",
           "items": {
-            "type": "number"
+            "type": "number",
+            "minimum": -1.0,
+            "maximum": 1.0
           },
-          "description": "Generalized IoU threshold for each class."
+          "description": "Generalized IoU threshold for each class.",
+          "default": [-0.1, -0.1, -0.1, -0.6, -0.6, -0.1, -0.1, -0.1]
+        }
+      },
+      "required": ["distance_threshold_list", "generalized_iou_threshold"],
+      "additionalProperties": false
+    }
+  },
+  "properties": {
+    "/**": {
+      "type": "object",
+      "properties": {
+        "ros__parameters": {
+          "$ref": "#/definitions/overlapped_judge"
         }
       },
-      "required": ["distance_threshold_list", "generalized_iou_threshold"]
+      "required": ["ros__parameters"],
+      "additionalProperties": false
     }
-  }
+  },
+  "required": ["/**"],
+  "additionalProperties": false
 }
diff --git a/perception/autoware_occupancy_grid_map_outlier_filter/README.md b/perception/autoware_occupancy_grid_map_outlier_filter/README.md
@@ -40,7 +40,7 @@ The following video is a sample. Yellow points are high occupancy probability, g
 
 ## Parameters
 
-{{ json_to_markdown("perception/occupancy_grid_map_outlier_filter/schema/occupancy_grid_map_outlier_filter.schema.json") }}
+{{ json_to_markdown("perception/autoware_occupancy_grid_map_outlier_filter/schema/occupancy_grid_map_outlier_filter.schema.json") }}
 
 ## Assumptions / Known limits
 
diff --git a/perception/autoware_probabilistic_occupancy_grid_map/README.md b/perception/autoware_probabilistic_occupancy_grid_map/README.md
@@ -21,12 +21,29 @@ You may need to choose `scan_origin_frame` and `gridmap_origin_frame` which mean
 
 ### Parameters
 
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/binary_bayes_filter_updater.schema.json") }}
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/grid_map.schema.json") }}
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/laserscan_based_occupancy_grid_map.schema.json") }}
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/multi_lidar_pointcloud_based_occupancy_grid_map.schema.json") }}
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/pointcloud_based_occupancy_grid_map.schema.json") }}
-{{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/synchronized_grid_map_fusion_node.schema.json") }}
+- binary bayes filter updater
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/binary_bayes_filter_updater.schema.json") }}
+
+- grid map
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/grid_map.schema.json") }}
+
+- laserscan based occupancy grid map
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/laserscan_based_occupancy_grid_map.schema.json") }}
+
+- multi lidar pointcloud based occupancy grid map
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/multi_lidar_pointcloud_based_occupancy_grid_map.schema.json") }}
+
+- pointcloud based occupancy grid map
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/pointcloud_based_occupancy_grid_map.schema.json") }}
+
+- synchronized grid map fusion
+
+  {{ json_to_markdown("perception/autoware_probabilistic_occupancy_grid_map/schema/synchronized_grid_map_fusion_node.schema.json") }}
 
 ### Downsample input pointcloud(Optional)
 
diff --git a/perception/autoware_probabilistic_occupancy_grid_map/schema/synchronized_grid_map_fusion_node.schema.json b/perception/autoware_probabilistic_occupancy_grid_map/schema/synchronized_grid_map_fusion_node.schema.json

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,8 @@`
`45`	`45`	`"remove_overlapped_unknown_objects",`
`46`	`46`	`"base_link_frame_id",`
`47`	`47`	`"priority_mode"`
`48`		`- ]`
	`48`	`+ ],`
	`49`	`+ "additionalProperties": false`
`49`	`50`	`}`
`50`	`51`	`},`
`51`	`52`	`"properties": {`
`@@ -56,8 +57,10 @@`
`56`	`57`	`"$ref": "#/definitions/object_association_merger"`
`57`	`58`	`}`
`58`	`59`	`},`
`59`		`- "required": ["ros__parameters"]`
	`60`	`+ "required": ["ros__parameters"],`
	`61`	`+ "additionalProperties": false`
`60`	`62`	`}`
`61`	`63`	`},`
`62`		`- "required": ["/**"]`
	`64`	`+ "required": ["/**"],`
	`65`	`+ "additionalProperties": false`
`63`	`66`	`}`