WIP: GeoPy tags

Author: idle-code

.envrc

@@ -0,0 +1,2 @@
+export PYTHONPATH="$PWD:$PYTHONPATH"
+export PATH="$PWD/.venv/bin:$PATH"

.idea/tempren.iml

@@ -19,7 +19,7 @@ repos:
     rev: '5.13.2'
     hooks:
     -    id: isort
-         language_version: python3.8
+         language_version: python3.11
          additional_dependencies:
          -   toml
 -   repo: https://github.com/awebdeveloper/pre-commit-stylelint
@@ -31,4 +31,4 @@ repos:
     rev: '24.8.0'
     hooks:
     -   id: black
-        language_version: python3.8
+        language_version: python3.11

.pre-commit-config.yaml

@@ -35,9 +35,10 @@ pymediainfo = {version = "^6.0.1", optional = true}
 isodate = "^0.6.1"
 gpxpy = "1.5.0"
 Pint = "^0.21.0"
+geopy = "^2.4.1"
 
-[tool.poetry.dev-dependencies]
-pytest = "^7.4.4"
+[tool.poetry.group.dev.dependencies]
+pytest = "^8.3.5"
 pytest-cov = "^4.1"
 coverage = "^7.2.7"
 pre-commit = "^3.5"

pyproject.toml

@@ -40,12 +40,15 @@ def _is_base_class(klass: type):
 
     def _visitor(module: ModuleType, klass: type):
         if not _is_base_class(klass):
+            log.debug(f"{klass} has no valid base ({base_klass})")
             return
+
         if module.__package__:
             category_name = CategoryName(module.__name__[len(module.__package__) + 1 :])
         else:
             category_name = CategoryName(module.__name__)
 
+        log.debug(f"Adding {klass} to {category_name} category")
         found_classes[category_name].append(klass)
 
     visit_types_in_package(package, _visitor)
@@ -80,4 +83,5 @@ def visit_types_in_module(
 
     for _, tag_class in inspect.getmembers(module):
         if isinstance(tag_class, type):
+            log.debug(f"Found tag {tag_class}")
             visitor(module, tag_class)

tempren/discovery.py

@@ -463,3 +463,13 @@ def process(self, file: File, context: Optional[str]) -> float:
         src_value = float(context) * self._src_unit
         dst_value = src_value.to(self._dst_unit)
         return dst_value.magnitude
+
+
+class NotTag(Tag):
+    """Negate truthfulness of provided context"""
+
+    require_context = True
+
+    def process(self, file: File, context: Optional[str]) -> bool:
+        assert context is not None
+        return not bool(context)

tempren/tags/core.py

@@ -148,14 +148,20 @@ def process(self, file: File, context: Optional[str]) -> Any:
         exif_dict = piexif.load(str(file.absolute_path))
         for src in exif_dict.values():
             if isinstance(src, dict) and self.tag_id in src:
-                return self._extract_value(src[self.tag_id])
+                return self._extract_value(self.tag_type, src[self.tag_id])
         else:
             raise MissingMetadataError()
 
-    def _extract_value(self, tag_value):
-        if self.tag_type in (TAG_TYPES.Rational, TAG_TYPES.SRational):
-            return round(tag_value[0] / tag_value[1], 1)
-        if self.tag_type == TAG_TYPES.Ascii:
+    def _extract_value(self, tag_type, tag_value):
+        if tag_type in (TAG_TYPES.Rational, TAG_TYPES.SRational):
+            if isinstance(tag_value[0], tuple):
+                return " ".join(str(self._extract_value(v)) for v in tag_value)
+            else:
+                if tag_value[1] == 1:
+                    return tag_value[0]
+                else:
+                    return round(tag_value[0] / tag_value[1], 1)
+        if tag_type == TAG_TYPES.Ascii:
             return tag_value.decode("ascii")
         return tag_value
 
@@ -187,3 +193,16 @@ def _generate_exif_tag_list() -> Iterator[str]:
 
 class ResolutionTagAlias(TagAlias):
     """%Image.Width()x%Image.Height()"""
+
+
+class GpsPositionTag(ExifTag):
+    """Latitude and longitude of place where photo was taken"""
+
+    def configure(self):
+        self.latitude_tag_id, self.latitude_tag_type = self._tag_name_to_id_type(
+            "GpsLatitude"
+        )
+
+
+class HasGpsPositionTagAlias(TagAlias):
+    """%Core.Not(){%Text.IsEmpty(){%Image.GpsPosition()}}"""

tempren/tags/geopy.py

@@ -209,6 +209,26 @@ def configure(self, separator: str = " "):  # type: ignore
         assert separator
         self.separator = separator
 
-    def process(self, file: File, context: Optional[str]) -> Any:
+    def process(self, file: File, context: Optional[str]) -> str:
         assert context is not None
         return self._pattern.sub("".join((r"\1", self.separator, r"\2")), context)
+
+
+class IsEmptyTag(Tag):
+    """Check if passed context is empty"""
+
+    require_context = True
+
+    def process(self, file: File, context: Optional[str]) -> bool:
+        assert context is not None
+        return len(str(context)) == 0
+
+
+class LengthTag(Tag):
+    """Number of characters in provided context"""
+
+    require_context = True
+
+    def process(self, file: File, context: Optional[str]) -> int:
+        assert context is not None
+        return len(str(context))