Add support for coordinates in Adobe format in XMP tags

mapillary_tools/exif_read.py

@@ -1,8 +1,10 @@
 import abc
 import datetime
 import logging
+import re
 import typing as T
 import xml.etree.ElementTree as et
+from fractions import Fraction
 from pathlib import Path
 
 import exifread
@@ -47,6 +49,34 @@ def gps_to_decimal(values: T.Tuple[Ratio, Ratio, Ratio]) -> T.Optional[float]:
     return degrees + minutes / 60 + seconds / 3600
 
 
+def parse_coordinate(coord: T.Optional[str]) -> T.Optional[float]:
+    """If the coordinate is in decimal degrees, just convert it to float,
+    otherwise try to parse it from the Adobe format
+    <degree,fractionalminute[NSEW]>
+    """
+
+    if not coord:
+        return None
+
+    try:
+        return float(coord)
+    except ValueError:
+        pass
+
+    adobe_format = re.match(r"(\d+),(\d{1,3}\.?\d*)([NSWE])", coord)
+    if adobe_format:
+        sign = {"N": 1, "S": -1, "E": 1, "W": -1}
+        deg = Ratio(int(adobe_format.group(1)), 1)
+        min_frac = Fraction.from_float(float(adobe_format.group(2)))
+        min = Ratio(min_frac.numerator, min_frac.denominator)
+        sec = Ratio(0, 1)
+        converted = gps_to_decimal((deg, min, sec))
+        if converted:
+            return converted * sign[adobe_format.group(3)]
+
+    return None
+
+
 def _parse_iso(dtstr: str) -> T.Optional[datetime.datetime]:
     try:
         return datetime.datetime.fromisoformat(dtstr)
@@ -378,20 +408,26 @@ def extract_direction(self) -> T.Optional[float]:
         )
 
     def extract_lon_lat(self) -> T.Optional[T.Tuple[float, float]]:
-        lat = self._extract_alternative_fields(["exif:GPSLatitude"], float)
+        lat_str: T.Optional[str] = self._extract_alternative_fields(
+            ["exif:GPSLatitude"], str
+        )
+        lat: T.Optional[float] = parse_coordinate(lat_str)
         if lat is None:
             return None
 
-        lon = self._extract_alternative_fields(["exif:GPSLongitude"], float)
+        lon_str: T.Optional[str] = self._extract_alternative_fields(
+            ["exif:GPSLongitude"], str
+        )
+        lon = parse_coordinate(lon_str)
         if lon is None:
             return None
 
         ref = self._extract_alternative_fields(["exif:GPSLongitudeRef"], str)
-        if ref and ref.upper() == "W":
+        if ref and ref.upper() == "W" and lon > 0:
             lon = -1 * lon
 
         ref = self._extract_alternative_fields(["exif:GPSLatitudeRef"], str)
-        if ref and ref.upper() == "S":
+        if ref and ref.upper() == "S" and lat > 0:
             lat = -1 * lat
 
         return lon, lat

mapillary_tools/exiftool_read.py

@@ -310,6 +310,10 @@ def extract_lon_lat(self) -> T.Optional[T.Tuple[float, float]]:
         if lon_lat is not None:
             return lon_lat
 
+        lon_lat = self._extract_lon_lat("XMP-exif:GPSLongitude", "XMP-exif:GPSLatitude")
+        if lon_lat is not None:
+            return lon_lat
+
         return None
 
     def _extract_lon_lat(

tests/integration/test_process.py

@@ -59,6 +59,16 @@
         "MAPDeviceModel": "VIRB 360",
         "MAPOrientation": 1,
     },
+    "adobe_coords.jpg": {
+        "filetype": "image",
+        "MAPLatitude": -0.0702668,
+        "MAPLongitude": 34.3819352,
+        "MAPCaptureTime": "2019_07_16_10_26_11_000",
+        "MAPCompassHeading": {"TrueHeading": 0, "MagneticHeading": 0},
+        "MAPDeviceMake": "SAMSUNG",
+        "MAPDeviceModel": "SM-C200",
+        "MAPOrientation": 1,
+    },
 }
 
 
@@ -260,6 +270,27 @@ def test_angle_with_offset_with_exiftool(setup_data: py.path.local):
     return test_angle_with_offset(setup_data, use_exiftool=True)
 
 
+def test_parse_adobe_coordinates(setup_data: py.path.local):
+    args = f"{EXECUTABLE} process --file_types=image {PROCESS_FLAGS} {setup_data}/adobe_coords"
+    x = subprocess.run(args, shell=True)
+    verify_descs(
+        [
+            {
+                "filename": str(Path(setup_data, "adobe_coords", "adobe_coords.jpg")),
+                "filetype": "image",
+                "MAPLatitude": -0.0702668,
+                "MAPLongitude": 34.3819352,
+                "MAPCaptureTime": _local_to_utc("2019-07-16T10:26:11"),
+                "MAPCompassHeading": {"TrueHeading": 0.0, "MagneticHeading": 0.0},
+                "MAPDeviceMake": "SAMSUNG",
+                "MAPDeviceModel": "SM-C200",
+                "MAPOrientation": 1,
+            }
+        ],
+        Path(setup_data, "adobe_coords/mapillary_image_description.json"),
+    )
+
+
 def test_zip(tmpdir: py.path.local, setup_data: py.path.local):
     zip_dir = tmpdir.mkdir("zip_dir")
     x = subprocess.run(

tests/integration/test_process_and_upload.py

@@ -181,7 +181,7 @@ def test_process_and_upload_images_only(
     setup_upload: py.path.local,
 ):
     x = subprocess.run(
-        f"{EXECUTABLE} --verbose process_and_upload --filetypes=image {UPLOAD_FLAGS} {PROCESS_FLAGS} {setup_data} {setup_data} {setup_data}/images/DSC00001.JPG --desc_path=-",
+        f"{EXECUTABLE} --verbose process_and_upload --filetypes=image {UPLOAD_FLAGS} {PROCESS_FLAGS} {setup_data}/images {setup_data}/images {setup_data}/images/DSC00001.JPG --desc_path=-",
         shell=True,
     )
     assert x.returncode == 0, x.stderr