Merge pull request #215 from SenteraLLC/fix/empty-layers

Fix/empty layers

Author: TrevorBurgoyne

CHANGELOG.md

@@ -6,6 +6,9 @@ All notable changes to this project will be documented here.
 
 Nothing yet.
 
+## [0.16.4] - May 2nd, 2025
+- Fix bug where empty polygon layers could be included in a `submit_button` hook payload.
+
 ## [0.16.3] - Apr 25th, 2025
 - Fix bug where the `Vanish` button wouldn't update its `locked` state when switching between subtasks.
 - Fix bug where clicking the `Vanish` button would unexpectedly set the size cookie, causing future annotations in the same subtask to be loaded in a very small size on subsequent sessions, with the only way to recover being to increase the annotation size.

demo/resume-from.html

@@ -172,7 +172,8 @@
                                 467.5,
                                 267.5
                             ]
-                        ]
+                        ],
+                        []
                     ],
                     "classification_payloads": [
                         {

dist/ulabel.js

@@ -1,7 +1,7 @@
 {
   "name": "ulabel",
   "description": "An image annotation tool.",
-  "version": "0.16.3",
+  "version": "0.16.4",
   "main": "dist/ulabel.js",
   "module": "dist/ulabel.js",
   "scripts": {

dist/ulabel.min.js

@@ -129,7 +129,7 @@ export class ULabelAnnotation {
                 const layer = this.spatial_payload[i];
                 // Ensure that the layer is an array
                 if (!Array.isArray(layer[0])) {
-                    console.log(`Layer ${i} points of id ${this.id} are not arrays. Removing layer.`);
+                    console.log(`Layer ${i} of id ${this.id} has an invalid or empty point array. Removing layer.`);
                     indices_to_remove.push(i);
                     continue;
                 }

package-lock.json

@@ -2484,8 +2484,15 @@ export class SubmitButtons extends ToolboxItem {
                     submit_payload["annotations"][stkey] = [];
 
                     // Add all of the annotations in that subtask
+                    let annotation: ULabelAnnotation;
                     for (let i = 0; i < ulabel.subtasks[stkey]["annotations"]["ordering"].length; i++) {
-                        const annotation = ulabel.subtasks[stkey]["annotations"]["access"][ulabel.subtasks[stkey]["annotations"]["ordering"][i]];
+                        try {
+                            annotation = ULabelAnnotation.from_json(ulabel.subtasks[stkey]["annotations"]["access"][ulabel.subtasks[stkey]["annotations"]["ordering"][i]]);
+                        } catch (e) {
+                            console.error("Error validating annotation during submit.", e);
+                            continue;
+                        }
+
                         // Skip any delete modes
                         if (
                             DELETE_MODES.includes(annotation.spatial_type)

package.json

@@ -1 +1 @@
-export const ULABEL_VERSION = "0.16.3";
+export const ULABEL_VERSION = "0.16.4";