From 6c07a49219cc11b6845787a9c80410d0bde7d077 Mon Sep 17 00:00:00 2001
From: Martin Turoci <martin.turoci@h2o.ai>
Date: Thu, 3 Nov 2022 10:40:19 +0100
Subject: [PATCH] chore: Rename polygon items to vertices. #1597

---
 py/h2o_wave/types.py                           | 18 +++++++++---------
 py/h2o_wave/ui.py                              |  6 +++---
 r/R/ui.R                                       |  8 ++++----
 .../resources/templates/wave-components.xml    |  8 ++++----
 tools/vscode-extension/component-snippets.json |  4 ++--
 ui/src/image_annotator.tsx                     | 18 +++++++++---------
 ui/src/image_annotator_polygon.ts              |  4 ++--
 7 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/py/h2o_wave/types.py b/py/h2o_wave/types.py
index 9668c5fc88..856c5361f2 100644
--- a/py/h2o_wave/types.py
+++ b/py/h2o_wave/types.py
@@ -6485,27 +6485,27 @@ class ImageAnnotatorPolygon:
     """
     def __init__(
             self,
-            items: List[ImageAnnotatorPoint],
+            vertices: List[ImageAnnotatorPoint],
     ):
-        _guard_vector('ImageAnnotatorPolygon.items', items, (ImageAnnotatorPoint,), False, False, False)
-        self.items = items
+        _guard_vector('ImageAnnotatorPolygon.vertices', vertices, (ImageAnnotatorPoint,), False, False, False)
+        self.vertices = vertices
         """List of points of the polygon."""
 
     def dump(self) -> Dict:
         """Returns the contents of this object as a dict."""
-        _guard_vector('ImageAnnotatorPolygon.items', self.items, (ImageAnnotatorPoint,), False, False, False)
+        _guard_vector('ImageAnnotatorPolygon.vertices', self.vertices, (ImageAnnotatorPoint,), False, False, False)
         return _dump(
-            items=[__e.dump() for __e in self.items],
+            vertices=[__e.dump() for __e in self.vertices],
         )
 
     @staticmethod
     def load(__d: Dict) -> 'ImageAnnotatorPolygon':
         """Creates an instance of this class using the contents of a dict."""
-        __d_items: Any = __d.get('items')
-        _guard_vector('ImageAnnotatorPolygon.items', __d_items, (dict,), False, False, False)
-        items: List[ImageAnnotatorPoint] = [ImageAnnotatorPoint.load(__e) for __e in __d_items]
+        __d_vertices: Any = __d.get('vertices')
+        _guard_vector('ImageAnnotatorPolygon.vertices', __d_vertices, (dict,), False, False, False)
+        vertices: List[ImageAnnotatorPoint] = [ImageAnnotatorPoint.load(__e) for __e in __d_vertices]
         return ImageAnnotatorPolygon(
-            items,
+            vertices,
         )
 
 
diff --git a/py/h2o_wave/ui.py b/py/h2o_wave/ui.py
index b59c29f5d7..835927a20e 100644
--- a/py/h2o_wave/ui.py
+++ b/py/h2o_wave/ui.py
@@ -2417,17 +2417,17 @@ def image_annotator_point(
 
 
 def image_annotator_polygon(
-        items: List[ImageAnnotatorPoint],
+        vertices: List[ImageAnnotatorPoint],
 ) -> ImageAnnotatorShape:
     """Create a polygon annotation shape.
 
     Args:
-        items: List of points of the polygon.
+        vertices: List of points of the polygon.
     Returns:
         A `h2o_wave.types.ImageAnnotatorPolygon` instance.
     """
     return ImageAnnotatorShape(polygon=ImageAnnotatorPolygon(
-        items,
+        vertices,
     ))
 
 
diff --git a/r/R/ui.R b/r/R/ui.R
index 38020a9b69..dfa3247536 100644
--- a/r/R/ui.R
+++ b/r/R/ui.R
@@ -2817,14 +2817,14 @@ ui_image_annotator_point <- function(
 
 #' Create a polygon annotation shape.
 #'
-#' @param items List of points of the polygon.
+#' @param vertices List of points of the polygon.
 #' @return A ImageAnnotatorPolygon instance.
 #' @export
 ui_image_annotator_polygon <- function(
-  items) {
-  .guard_vector("items", "WaveImageAnnotatorPoint", items)
+  vertices) {
+  .guard_vector("vertices", "WaveImageAnnotatorPoint", vertices)
   .o <- list(polygon=list(
-    items=items))
+    vertices=vertices))
   class(.o) <- append(class(.o), c(.wave_obj, "WaveImageAnnotatorShape"))
   return(.o)
 }
diff --git a/tools/intellij-plugin/src/main/resources/templates/wave-components.xml b/tools/intellij-plugin/src/main/resources/templates/wave-components.xml
index a70b63376e..b88c9dd5a2 100644
--- a/tools/intellij-plugin/src/main/resources/templates/wave-components.xml
+++ b/tools/intellij-plugin/src/main/resources/templates/wave-components.xml
@@ -230,8 +230,8 @@
       <option name="Python" value="true"/>
     </context>
   </template>
-  <template name="w_image_annotator_polygon" value="ui.image_annotator_polygon(items=[&#10;	$items$	&#10;]),$END$" description="Create a minimal Wave ImageAnnotatorPolygon." toReformat="true" toShortenFQNames="true">
-    <variable name="items" expression="" defaultValue="" alwaysStopAt="true"/>
+  <template name="w_image_annotator_polygon" value="ui.image_annotator_polygon(vertices=[&#10;	$vertices$	&#10;]),$END$" description="Create a minimal Wave ImageAnnotatorPolygon." toReformat="true" toShortenFQNames="true">
+    <variable name="vertices" expression="" defaultValue="" alwaysStopAt="true"/>
     <context>
       <option name="Python" value="true"/>
     </context>
@@ -1416,8 +1416,8 @@
       <option name="Python" value="true"/>
     </context>
   </template>
-  <template name="w_full_image_annotator_polygon" value="ui.image_annotator_polygon(items=[&#10;	$items$	&#10;]),$END$" description="Create Wave ImageAnnotatorPolygon with full attributes." toReformat="true" toShortenFQNames="true">
-    <variable name="items" expression="" defaultValue="" alwaysStopAt="true"/>
+  <template name="w_full_image_annotator_polygon" value="ui.image_annotator_polygon(vertices=[&#10;	$vertices$	&#10;]),$END$" description="Create Wave ImageAnnotatorPolygon with full attributes." toReformat="true" toShortenFQNames="true">
+    <variable name="vertices" expression="" defaultValue="" alwaysStopAt="true"/>
     <context>
       <option name="Python" value="true"/>
     </context>
diff --git a/tools/vscode-extension/component-snippets.json b/tools/vscode-extension/component-snippets.json
index 7935d332a1..2620329a4c 100644
--- a/tools/vscode-extension/component-snippets.json
+++ b/tools/vscode-extension/component-snippets.json
@@ -240,7 +240,7 @@
   "Wave ImageAnnotatorPolygon": {
     "prefix": "w_image_annotator_polygon",
     "body": [
-      "ui.image_annotator_polygon(items=[\n\t\t$1\t\t\n]),$0"
+      "ui.image_annotator_polygon(vertices=[\n\t\t$1\t\t\n]),$0"
     ],
     "description": "Create a minimal Wave ImageAnnotatorPolygon."
   },
@@ -1185,7 +1185,7 @@
   "Wave Full ImageAnnotatorPolygon": {
     "prefix": "w_full_image_annotator_polygon",
     "body": [
-      "ui.image_annotator_polygon(items=[\n\t\t$1\t\t\n]),$0"
+      "ui.image_annotator_polygon(vertices=[\n\t\t$1\t\t\n]),$0"
     ],
     "description": "Create a full Wave ImageAnnotatorPolygon."
   },
diff --git a/ui/src/image_annotator.tsx b/ui/src/image_annotator.tsx
index fbd11179a8..ac78e140ad 100644
--- a/ui/src/image_annotator.tsx
+++ b/ui/src/image_annotator.tsx
@@ -19,7 +19,7 @@ export interface ImageAnnotatorPoint {
 /** Create a polygon annotation shape. */
 export interface ImageAnnotatorPolygon {
   /** List of points of the polygon. */
-  items: ImageAnnotatorPoint[]
+  vertices: ImageAnnotatorPoint[]
 }
 
 /** Create a rectangular annotation shape. */
@@ -113,7 +113,7 @@ const
   eventToCursor = (event: React.MouseEvent, rect: DOMRect) => ({ cursor_x: event.clientX - rect.left, cursor_y: event.clientY - rect.top }),
   getIntersectedShape = (shapes: DrawnShape[], cursor_x: F, cursor_y: F) => shapes.find(({ shape, isFocused }) => {
     if (shape.rect) return isIntersectingRect(cursor_x, cursor_y, shape.rect, isFocused)
-    if (shape.polygon) return isIntersectingPolygon({ x: cursor_x, y: cursor_y }, shape.polygon.items, isFocused)
+    if (shape.polygon) return isIntersectingPolygon({ x: cursor_x, y: cursor_y }, shape.polygon.vertices, isFocused)
   }),
   getCorrectCursor = (cursor_x: U, cursor_y: U, focused?: DrawnShape, intersected?: DrawnShape, isSelect = false) => {
     let cursor = intersected
@@ -123,8 +123,8 @@ const
         : 'crosshair'
     if (intersected?.isFocused && intersected.shape.rect) cursor = getRectCornerCursor(intersected.shape.rect, cursor_x, cursor_y) || 'move'
     else if (focused?.shape.rect) cursor = getRectCornerCursor(focused.shape.rect, cursor_x, cursor_y) || cursor
-    else if (intersected?.isFocused && intersected.shape.polygon) cursor = getPolygonPointCursor(intersected.shape.polygon.items, cursor_x, cursor_y) || 'move'
-    else if (focused?.shape.polygon) cursor = getPolygonPointCursor(focused.shape.polygon.items, cursor_x, cursor_y) || cursor
+    else if (intersected?.isFocused && intersected.shape.polygon) cursor = getPolygonPointCursor(intersected.shape.polygon.vertices, cursor_x, cursor_y) || 'move'
+    else if (focused?.shape.polygon) cursor = getPolygonPointCursor(focused.shape.polygon.vertices, cursor_x, cursor_y) || cursor
 
     return cursor
   },
@@ -140,7 +140,7 @@ const
         }
       }
     }
-    else if (shape.polygon) return { tag, shape: { polygon: { items: shape.polygon.items.map(i => ({ x: i.x * aspectRatio, y: i.y * aspectRatio })) } } }
+    else if (shape.polygon) return { tag, shape: { polygon: { vertices: shape.polygon.vertices.map(i => ({ x: i.x * aspectRatio, y: i.y * aspectRatio })) } } }
     return { tag, shape }
   })
 
@@ -172,7 +172,7 @@ export const XImageAnnotator = ({ model }: { model: ImageAnnotator }) => {
       setDrawnShapes(shapes => {
         shapes.forEach(({ shape, isFocused, tag }) => {
           if (shape.rect) rectRef.current?.drawRect(shape.rect, getCurrentTagColor(tag), isFocused)
-          else if (shape.polygon) polygonRef.current?.drawPolygon(shape.polygon.items, getCurrentTagColor(tag), true, isFocused)
+          else if (shape.polygon) polygonRef.current?.drawPolygon(shape.polygon.vertices, getCurrentTagColor(tag), true, isFocused)
         })
         return shapes
       })
@@ -260,13 +260,13 @@ export const XImageAnnotator = ({ model }: { model: ImageAnnotator }) => {
         }
         case 'select': {
           if (intersected) setActiveTag(intersected.tag)
-          if (intersected?.shape.polygon) polygonRef.current?.addAuxPoint(cursor_x, cursor_y, intersected.shape.polygon.items)
+          if (intersected?.shape.polygon) polygonRef.current?.addAuxPoint(cursor_x, cursor_y, intersected.shape.polygon.vertices)
           polygonRef.current?.resetDragging()
 
           setDrawnShapes(drawnShapes => drawnShapes.map(s => {
             s.isFocused = s === intersected
             if (s.isFocused && s.shape.polygon && polygonRef.current) {
-              s.shape.polygon.items = polygonRef.current.getPolygonPointsWithAux(s.shape.polygon.items)
+              s.shape.polygon.vertices = polygonRef.current.getPolygonPointsWithAux(s.shape.polygon.vertices)
             }
             return s
           }))
@@ -342,7 +342,7 @@ export const XImageAnnotator = ({ model }: { model: ImageAnnotator }) => {
         tag,
         shape: {
           polygon: {
-            items: shape.polygon.items
+            vertices: shape.polygon.vertices
               .filter((i: DrawnPoint) => !i.isAux)
               .map(i => ({ x: i.x / aspectRatio, y: i.y / aspectRatio }))
           }
diff --git a/ui/src/image_annotator_polygon.ts b/ui/src/image_annotator_polygon.ts
index b4af050b9d..1f555318fa 100644
--- a/ui/src/image_annotator_polygon.ts
+++ b/ui/src/image_annotator_polygon.ts
@@ -39,7 +39,7 @@ export class PolygonAnnotator {
       return
     }
 
-    const clickedPolygonPoint = focused.shape.polygon.items.find(p => isIntersectingPoint(p, cursor_x, cursor_y))
+    const clickedPolygonPoint = focused.shape.polygon.vertices.find(p => isIntersectingPoint(p, cursor_x, cursor_y))
     this.draggedPoint = clickedPolygonPoint || this.draggedPoint
     if (this.draggedPoint) {
       this.draggedPoint.x += cursor_x - this.draggedPoint.x
@@ -47,7 +47,7 @@ export class PolygonAnnotator {
     }
     else if (intersected == focused || this.draggedShape) {
       this.draggedShape = intersected?.shape.polygon && intersected.isFocused ? intersected : this.draggedShape
-      this.draggedShape?.shape.polygon?.items.forEach(p => {
+      this.draggedShape?.shape.polygon?.vertices.forEach(p => {
         p.x += cursor_x - clickStartPosition!.x
         p.y += cursor_y - clickStartPosition!.y
       })