From edef764b3eee13fd47b2cf31d2baa2684ad35ad6 Mon Sep 17 00:00:00 2001
From: Roman Donchenko <roman@cvat.ai>
Date: Tue, 26 Nov 2024 18:21:50 +0200
Subject: [PATCH] Rename `FunctionCallRequestSerializer.convMaskToPoly` to fit
 the naming convention (#8743)

Keep the old name as a compatibility alias for the time being.
---
 .../20241126_140417_roman_rename_conv_mask_to_poly.md | 11 +++++++++++
 .../controls-side-bar/tools-control.tsx               |  4 ++--
 .../components/model-runner-modal/detector-runner.tsx |  4 ++--
 cvat/apps/lambda_manager/serializers.py               |  3 ++-
 cvat/apps/lambda_manager/views.py                     |  2 +-
 cvat/schema.yml                                       |  6 +++++-
 6 files changed, 23 insertions(+), 7 deletions(-)
 create mode 100644 changelog.d/20241126_140417_roman_rename_conv_mask_to_poly.md

diff --git a/changelog.d/20241126_140417_roman_rename_conv_mask_to_poly.md b/changelog.d/20241126_140417_roman_rename_conv_mask_to_poly.md
new file mode 100644
index 000000000000..1788c1fe6c41
--- /dev/null
+++ b/changelog.d/20241126_140417_roman_rename_conv_mask_to_poly.md
@@ -0,0 +1,11 @@
+### Added
+
+- The `POST /api/lambda/requests` endpoint now has a `conv_mask_to_poly`
+  parameter with the same semantics as the old `convMaskToPoly` parameter
+  (<https://github.com/cvat-ai/cvat/pull/8743>)
+
+### Deprecated
+
+- The `convMaskToPoly` parameter of the `POST /api/lambda/requests` endpoint
+  is deprecated; use `conv_mask_to_poly` instead
+  (<https://github.com/cvat-ai/cvat/pull/8743>)
diff --git a/cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx b/cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx
index a31307277e68..dc73360d1f1d 100644
--- a/cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx
+++ b/cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx
@@ -1254,8 +1254,8 @@ export class ToolsControlComponent extends React.PureComponent<Props, State> {
                     try {
                         this.setState({ mode: 'detection', fetching: true });
 
-                        // The function call endpoint doesn't support the cleanup and convMaskToPoly parameters.
-                        const { cleanup, convMaskToPoly, ...restOfBody } = body;
+                        // The function call endpoint doesn't support the cleanup and conv_mask_to_poly parameters.
+                        const { cleanup, conv_mask_to_poly: convMaskToPoly, ...restOfBody } = body;
 
                         const result = await core.lambda.call(jobInstance.taskId, model, {
                             ...restOfBody, frame, job: jobInstance.id,
diff --git a/cvat-ui/src/components/model-runner-modal/detector-runner.tsx b/cvat-ui/src/components/model-runner-modal/detector-runner.tsx
index d6c92826b662..ab3393b2c290 100644
--- a/cvat-ui/src/components/model-runner-modal/detector-runner.tsx
+++ b/cvat-ui/src/components/model-runner-modal/detector-runner.tsx
@@ -40,7 +40,7 @@ type ServerMapping = Record<string, {
 export interface DetectorRequestBody {
     mapping: ServerMapping;
     cleanup: boolean;
-    convMaskToPoly: boolean;
+    conv_mask_to_poly: boolean;
 }
 
 function convertMappingToServer(mapping: FullMapping): ServerMapping {
@@ -235,7 +235,7 @@ function DetectorRunner(props: Props): JSX.Element {
                                 runInference(model, {
                                     mapping: serverMapping,
                                     cleanup,
-                                    convMaskToPoly: convertMasksToPolygons,
+                                    conv_mask_to_poly: convertMasksToPolygons,
                                 });
                             } else if (model.kind === ModelKind.REID) {
                                 runInference(model, { threshold, max_distance: distance });
diff --git a/cvat/apps/lambda_manager/serializers.py b/cvat/apps/lambda_manager/serializers.py
index 05049a7aeae8..ab8809bd7cc8 100644
--- a/cvat/apps/lambda_manager/serializers.py
+++ b/cvat/apps/lambda_manager/serializers.py
@@ -27,7 +27,8 @@ class FunctionCallRequestSerializer(serializers.Serializer):
     max_distance = serializers.IntegerField(required=False)
     threshold = serializers.FloatField(required=False)
     cleanup = serializers.BooleanField(help_text="Whether existing annotations should be removed", default=False)
-    convMaskToPoly = serializers.BooleanField(default=False) # TODO: use lowercase naming
+    convMaskToPoly = serializers.BooleanField(required=False, source="conv_mask_to_poly", write_only=True, help_text="Deprecated; use conv_mask_to_poly instead")
+    conv_mask_to_poly = serializers.BooleanField(required=False, help_text="Convert mask shapes to polygons")
     mapping = serializers.DictField(child=LabelMappingEntrySerializer(), required=False,
         help_text="Label mapping from the model to the task labels"
     )
diff --git a/cvat/apps/lambda_manager/views.py b/cvat/apps/lambda_manager/views.py
index c5401792c354..559ef29813b5 100644
--- a/cvat/apps/lambda_manager/views.py
+++ b/cvat/apps/lambda_manager/views.py
@@ -1163,7 +1163,7 @@ def create(self, request):
             task = request_data['task']
             job = request_data.get('job', None)
             cleanup = request_data.get('cleanup', False)
-            conv_mask_to_poly = request_data.get('convMaskToPoly', False)
+            conv_mask_to_poly = request_data.get('conv_mask_to_poly', False)
             mapping = request_data.get('mapping')
             max_distance = request_data.get('max_distance')
         except KeyError as err:
diff --git a/cvat/schema.yml b/cvat/schema.yml
index ea7ca03352e1..b8d02651ffc6 100644
--- a/cvat/schema.yml
+++ b/cvat/schema.yml
@@ -8060,7 +8060,11 @@ components:
           description: Whether existing annotations should be removed
         convMaskToPoly:
           type: boolean
-          default: false
+          writeOnly: true
+          description: Deprecated; use conv_mask_to_poly instead
+        conv_mask_to_poly:
+          type: boolean
+          description: Convert mask shapes to polygons
         mapping:
           type: object
           additionalProperties: