Fixed detected_text to use image orientation if available

osxphotos/_version.py

@@ -1,3 +1,3 @@
 """ version info """
 
-__version__ = "0.42.82"
+__version__ = "0.42.83"

osxphotos/photoinfo/photoinfo.py

@@ -1054,15 +1054,15 @@ class PhotoInfo:
             return self._info["orientation"]
 
         # For Photos 5+, try to get the adjusted orientation
-        if self.hasadjustments:
-            if self.adjustments:
-                return self.adjustments.adj_orientation
-            else:
-                # can't reliably determine orientation for edited photo if adjustmentinfo not available
-                return 0
-        else:
+        if not self.hasadjustments:
             return self._info["orientation"]
 
+        if self.adjustments:
+            return self.adjustments.adj_orientation
+        else:
+            # can't reliably determine orientation for edited photo if adjustmentinfo not available
+            return 0
+
     @property
     def original_height(self):
         """returns height of the original photo version in pixels"""
@@ -1156,7 +1156,8 @@ class PhotoInfo:
         md = OSXMetaData(path)
         detected_text = md.get_attribute("osxphotos_detected_text")
         if detected_text is None:
-            detected_text = detect_text(path)
+            orientation = self.orientation or None
+            detected_text = detect_text(path, orientation)
             md.set_attribute("osxphotos_detected_text", detected_text)
         return detected_text
 

osxphotos/text_detection.py

@@ -1,7 +1,7 @@
 """ Use Apple's Vision Framework via PyObjC to perform text detection on images (macOS 10.15+ only) """
 
 import logging
-from typing import List
+from typing import List, Optional
 
 import objc
 import Quartz
@@ -22,8 +22,13 @@ else:
     vision = True
 
 
-def detect_text(img_path: str) -> List:
-    """process image at img_path with VNRecognizeTextRequest and return list of results"""
+def detect_text(img_path: str, orientation: Optional[int] = None) -> List:
+    """process image at img_path with VNRecognizeTextRequest and return list of results
+
+    Args:
+        img_path: path to the image file
+        orientation: optional EXIF orientation (if known, passing orientation may improve quality of results)
+    """
     if not vision:
         logging.warning(f"detect_text not implemented for this version of macOS")
         return []
@@ -40,9 +45,18 @@ def detect_text(img_path: str) -> List:
             input_image = Quartz.CIImage.imageWithContentsOfURL_(input_url)
 
         vision_options = NSDictionary.dictionaryWithDictionary_({})
-        vision_handler = Vision.VNImageRequestHandler.alloc().initWithCIImage_options_(
-            input_image, vision_options
-        )
+        if orientation is not None:
+            if not 1 <= orientation <= 8:
+                raise ValueError("orientation must be between 1 and 8")
+            vision_handler = Vision.VNImageRequestHandler.alloc().initWithCIImage_orientation_options_(
+                input_image, orientation, vision_options
+            )
+        else:
+            vision_handler = (
+                Vision.VNImageRequestHandler.alloc().initWithCIImage_options_(
+                    input_image, vision_options
+                )
+            )
         results = []
         handler = make_request_handler(results)
         vision_request = (