@meta-sam/graphics 0.1.6 → 0.1.7

package/README.md

@@ -153,8 +153,7 @@ complete current view to every `update()` call.
 
 - `update()` calls are serialized in invocation order. Always await the update that
   must be visible before calling `render()`.
-- Masks and boxes are retained. Text and point records are accepted but are not
-  drawn.
+- Masks and boxes are retained. Text records are accepted but are not drawn.
 - Mask records are collapsed by `identity`; the greatest per-mask `revision` wins.
   At the same revision, conflicting mask data is rejected.
 - Boxes are collapsed by frame and `objectId`; the last box in the accepted view
@@ -174,7 +173,8 @@ collapsed.
 
 Graphics accepts only complete masks with this shape, where `encoding` is
 `lossless` (the SAM API default, payloads starting with `~`) or `one_bit`
-(payloads starting with `!`):
+(payloads starting with `!`) and `payload` is the opaque base85 text from the
+wire, passed through unchanged:
 
 ```ts
 {

package/dist/renderer.js

@@ -52,8 +52,6 @@ export function objectColor(objectId) {
  */
 const RETAINED_MASK_OVERHEAD = 64;
 function sameBounds(left, right) {
-    if (left === undefined || right === undefined)
-        return left === right;
     return (left.left === right.left &&
         left.top === right.top &&
         left.right === right.right &&
@@ -150,7 +148,7 @@ function retainMask(record) {
         ...(record.frame === undefined ? {} : { frameIndex: record.frame.frameIndex }),
         width: record.mask.width,
         height: record.mask.height,
-        ...(record.bounds === undefined ? {} : { bounds: record.bounds }),
+        bounds: record.bounds,
     };
 }
 function traceMask(mask, limit) {
@@ -375,22 +373,18 @@ export class SegmentationRenderer {
                     continue;
                 const color = colorFor(mask.objectId);
                 context.fillStyle = color;
-                if (mask.bounds === undefined) {
-                    this.#paintMask(context, traced.path, color, outlineWidth);
+                // The raster covers the record's box: scale width × height into bounds.
+                context.save();
+                try {
+                    const boundsScaleX = (mask.bounds.right - mask.bounds.left) / mask.width;
+                    const boundsScaleY = (mask.bounds.bottom - mask.bounds.top) / mask.height;
+                    context.translate(mask.bounds.left, mask.bounds.top);
+                    context.scale(boundsScaleX, boundsScaleY);
+                    this.#paintMask(context, traced.path, color, outlineWidth /
+                        Math.max(Math.abs(boundsScaleX), Math.abs(boundsScaleY), Number.MIN_VALUE));
                 }
-                else {
-                    context.save();
-                    try {
-                        const boundsScaleX = (mask.bounds.right - mask.bounds.left) / mask.width;
-                        const boundsScaleY = (mask.bounds.bottom - mask.bounds.top) / mask.height;
-                        context.translate(mask.bounds.left, mask.bounds.top);
-                        context.scale(boundsScaleX, boundsScaleY);
-                        this.#paintMask(context, traced.path, color, outlineWidth /
-                            Math.max(Math.abs(boundsScaleX), Math.abs(boundsScaleY), Number.MIN_VALUE));
-                    }
-                    finally {
-                        context.restore();
-                    }
+                finally {
+                    context.restore();
                 }
             }
             for (const box of state.boxes.values()) {
@@ -555,13 +549,16 @@ export class SegmentationRenderer {
             mask.height <= 0) {
             throw new InvalidMaskPayloadError('Mask dimensions must be positive safe integers.');
         }
-        if (record.bounds !== undefined) {
-            const { left, top, right, bottom } = record.bounds;
-            if (![left, top, right, bottom].every(Number.isFinite) ||
-                right <= left ||
-                bottom <= top) {
-                throw new InvalidMaskPayloadError('Mask bounds are invalid.');
-            }
+        if (record.bounds === undefined ||
+            ![
+                record.bounds.left,
+                record.bounds.top,
+                record.bounds.right,
+                record.bounds.bottom,
+            ].every(Number.isFinite) ||
+            record.bounds.right <= record.bounds.left ||
+            record.bounds.bottom <= record.bounds.top) {
+            throw new InvalidMaskPayloadError('Mask bounds are invalid.');
         }
         const area = mask.width * mask.height;
         if (!Number.isSafeInteger(area) || area > this.#limits.maxMaskArea) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meta-sam/graphics",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Retained Canvas 2D renderer for SAM 3 segmentation",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://github.com/meta-models/meta-sam#readme",
@@ -33,7 +33,7 @@
     "node": "^20.17.0 || >=22.9.0"
   },
   "dependencies": {
-    "@meta-sam/parser": "0.0.9"
+    "@meta-sam/parser": "0.0.10"
   },
   "scripts": {
     "build": "tsc -b",