fix: fixes after review

Author: aianch

config_library/pattern-2/default/config.yaml

@@ -605,7 +605,7 @@ summarization:
 assessment:
   image:
     target_width: '951'
-    target_height: 1268
+    target_height: '1268'
   default_confidence_threshold: '0.9'
   top_p: '0.1'
   max_tokens: '10000'

lib/idp_common_pkg/idp_common/classification/service.py

@@ -608,7 +608,7 @@ def classify_page_bedrock(
         # Load image content from URI with configurable dimensions
         if image_uri:
             try:
-                image_config = self.config.get("image", {})
+                image_config = self.config.get("classification", {}).get("image", {})
                 target_width = image_config.get("target_width")
                 target_height = image_config.get("target_height")
 

lib/idp_common_pkg/tests/unit/classification/test_classification_service.py

@@ -232,7 +232,7 @@ def test_classify_page_bedrock_success(
 
         # Verify calls
         mock_get_text.assert_called_once_with("s3://bucket/text.txt")
-        mock_prepare_image.assert_called_once_with("s3://bucket/image.jpg", 951, 1268)
+        mock_prepare_image.assert_called_once_with("s3://bucket/image.jpg")
         mock_prepare_bedrock_image.assert_called_once_with(b"image_data")
         mock_invoke.assert_called_once()
 

patterns/pattern-2/template.yaml

@@ -147,14 +147,14 @@ Resources:
                     description: "Target image width in pixels. Images larger than this will be resized. Default: 951"
                     default: 951
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 0
                   target_height:
                     type: number
                     description: "Target image height in pixels. Images larger than this will be resized. Default: 1268"
                     default: 1268
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 1
               features:
                   type: array
@@ -402,14 +402,14 @@ Resources:
                     description: "Target image width in pixels. Images larger than this will be resized. Default: 951"
                     default: 951
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 0
                   target_height:
                     type: number
                     description: "Target image height in pixels. Images larger than this will be resized. Default: 1268"
                     default: 1268
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 1
               model:
                 type: string
@@ -485,14 +485,14 @@ Resources:
                     description: "Target image width in pixels. Images larger than this will be resized. Default: 951"
                     default: 951
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 0
                   target_height:
                     type: number
                     description: "Target image height in pixels. Images larger than this will be resized. Default: 1268"
                     default: 1268
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 1
               model:
                 type: string
@@ -554,14 +554,14 @@ Resources:
                     description: "Target image width in pixels. Images larger than this will be resized. Default: 951"
                     default: 951
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 0
                   target_height:
                     type: number
                     description: "Target image height in pixels. Images larger than this will be resized. Default: 1268"
                     default: 1268
                     minimum: 100
-                    maximum: 4000
+                    maximum: 3500
                     order: 1
               default_confidence_threshold:
                 type: number