geopic-tag-reader 1.0.5__tar.gz → 1.1.0__tar.gz

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/CHANGELOG.md

@@ -1,4 +1,5 @@
 # Changelog
+
 All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
@@ -6,137 +7,162 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.1.0] - 2024-04-17
+
+### Changed
+
+- Encoding information (`charset=...`) is now stripped out of text EXIF tags.
+- `GeoPicTags` objects returns `ts` as a Python `datetime` object (instead of decimal epoch).
+- Improved timezone handling in reader, GPS coordinates are used to find appropriate timezone when no timezone is defined in EXIF tags. If a UTC fallback is done, a warning is thrown.
+
+## [1.0.6] - 2024-04-02
+
+### Changed
+
+- Bump some dependencies, the most important one being [typer](https://typer.tiangolo.com/) since it removes the need for typer-cli.
 
 ## [1.0.5] - 2024-03-06
 
 ### Fixed
+
 - Automatic detection of 360° pictures based on make and model also checks for image dimensions (to avoid false positive, when 360° camera can also take flat pictures).
 - Slash character in GPS date stamp was not correctly interpreted.
 
-
 ## [1.0.4] - 2024-02-12
 
 ### Changed
-- When using date/time not coming from GPS, offset time EXIF field are also read (`Exif.Photo.OffsetTimeOriginal` or `Exif.Photo.OffsetTime` depending on used date/time EXIF tag).
 
+- When using date/time not coming from GPS, offset time EXIF field are also read (`Exif.Photo.OffsetTimeOriginal` or `Exif.Photo.OffsetTime` depending on used date/time EXIF tag).
 
 ## [1.0.3] - 2023-12-18
 
 ### Added
+
 - Support reading date from EXIF fields `Exif.Image.DateTime` and `Xmp.GPano.SourceImageCreateTime`.
 - Auto-detect if a picture is 360° based on make and model.
 
 ### Fixed
-- Avoid failures for pictures with invalid offset for maker notes (due to PyExiv2 log level).
 
+- Avoid failures for pictures with invalid offset for maker notes (due to PyExiv2 log level).
 
 ## [1.0.2] - 2023-11-20
 
 ### Added
+
 - A warning is thrown when microseconds values between decimal seconds and sub-second time fields are not matching.
 
 ### Changed
-- Fraction values for date, time and GPS coordinates are supported.
 
+- Fraction values for date, time and GPS coordinates are supported.
 
 ## [1.0.1] - 2023-11-17
 
 ### Fixed
-- `DateTimeOriginal` field wasn't correctly read when seconds value was decimal.
 
+- `DateTimeOriginal` field wasn't correctly read when seconds value was decimal.
 
 ## [1.0.0] - 2023-10-18
 
 ### Added
+
 - Add subseconds to written metadata
 - Read altitude in metadata
 - Write direction and altitude metadata
 - Add `additional_exif` tags in `writePictureMetadata`
 
 ### Changed
-- EXIF list of tags now uses the [Exiv2](https://exiv2.org/metadata.html) notation (example `Exif.Image.Artist`) in returned data. To do this, pyexiv2 dependency is always necessary, and Pillow dependency has been removed. As a consequence, `readPictureMetadata` function __now takes in input `bytes`__ read from picture file instead of Pillow image. This is a breaking change.
-- Use overriden metadata is available to localize overriden capture_time
 
+- EXIF list of tags now uses the [Exiv2](https://exiv2.org/metadata.html) notation (example `Exif.Image.Artist`) in returned data. To do this, pyexiv2 dependency is always necessary, and Pillow dependency has been removed. As a consequence, `readPictureMetadata` function **now takes in input `bytes`** read from picture file instead of Pillow image. This is a breaking change.
+- Use overriden metadata is available to localize overriden capture_time
 
 ## [0.4.1] - 2023-09-08
 
 ### Added
-- Latitude and longitude values are checked to verify they fit into WGS84 projection bounds (-180, -90, 180, 90).
 
+- Latitude and longitude values are checked to verify they fit into WGS84 projection bounds (-180, -90, 180, 90).
 
 ## [0.4.0] - 2023-09-01
 
 ### Added
+
 - When a picture does not contain a mandatory exif tag (coordinates or datetime), a `PartialExifException` is thrown containing some information about what has been parsed and what is missing.
 
 ## [0.3.1] - 2023-07-31
 
 ### Added
+
 - A way to write exif lon/lat and type tags.
 
 ## [0.3.0] - 2023-07-31
 
 ### Added
+
 - Support of any date/time separator for EXIF tag `DateTimeOriginal`
 - A way to write exif tags. To use this, you need to install this library with the extra `[write-exif]`.
 
-
 ## [0.2.0] - 2023-07-13
 
 ### Added
+
 - Support of cropped equirectangular panoramas
 
 ### Changed
+
 - Support python 3.8
 
 ## [0.1.3]
 
 ### Changed
+
 - Bump [Typer](typer.tiangolo.com/) version, and use fork of [Typer-cli](https://gitlab.com/geovisio/infra/typer-cli)
 
 ## [0.1.2]
 
 ### Added
-- Full typing support ([PEP 484](https://peps.python.org/pep-0484/) and [PEP 561](https://peps.python.org/pep-0561/))
 
+- Full typing support ([PEP 484](https://peps.python.org/pep-0484/) and [PEP 561](https://peps.python.org/pep-0561/))
 
 ## [0.1.1]
 
 ### Added
-- Support of Mapillary tags stored in EXIF tag `ImageDescription`
 
+- Support of Mapillary tags stored in EXIF tag `ImageDescription`
 
 ## [0.1.0]
 
 ### Added
+
 - If GPS Date or time can't be read, fallbacks to Original Date EXIF tag associated with a reader warning
 - New EXIF tags are supported: `GPSDateTime`
 
 ### Changed
+
 - `tag_reader:warning` property has been moved from EXIF, and is now available as a direct property named `tagreader_warnings` of `GeoPicTags` class
 - Reader now supports `GPSLatitude` and `GPSLongitude` stored as decimal values instead of tuple
 - Reader now supports reading `FocalLength` written in `NUMBER/NUMBER` format
 - If EXIF tags for heading `PoseHeadingDegrees` and `GPSImgDirection` have contradicting values, we use by default `GPSImgDirection` value and issue a warning, instead of raising an error
 
 ### Fixed
-- EXIF tag `SubsecTimeOriginal` was not correctly read due to a typo
 
+- EXIF tag `SubsecTimeOriginal` was not correctly read due to a typo
 
 ## [0.0.2] - 2023-05-10
 
 ### Added
+
 - EXIF tag `UserComment` is now read and available in raw `exif` tags
 - If not set, `GPSLatitudeRef` defaults to North and `GPSLongitudeRef` defaults to East
 - A new `tag_reader:warning` property lists non-blocking warnings raised while reading EXIF tags
 
-
 ## [0.0.1] - 2023-03-31
 
 ### Added
-- EXIF tag reading methods extracted from [GeoVisio API](https://gitlab.com/geovisio/api)
 
+- EXIF tag reading methods extracted from [GeoVisio API](https://gitlab.com/geovisio/api)
 
-[Unreleased]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.5...main
+[Unreleased]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.1.0...main
+[1.1.0]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.6...1.1.0
+[1.0.6]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.5...1.0.6
 [1.0.5]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.4...1.0.5
 [1.0.4]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.3...1.0.4
 [1.0.3]: https://gitlab.com/geovisio/geo-picture-tag-reader/-/compare/1.0.2...1.0.3

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/Makefile

@@ -14,7 +14,7 @@ fmt: ## Format code
 ci: type-check fmt test ## Run all check like the ci
 
 docs:  ## Generates documentation from Typer embedded docs
-	python -m typer_cli ./geopic_tag_reader/main.py utils docs --name geopic-tag-reader --output docs/CLI_USAGE.md
+	python -m typer ./geopic_tag_reader/main.py utils docs --name geopic-tag-reader --output docs/CLI_USAGE.md
 	lazydocs --overview-file API_USAGE.md --ignored-modules main --output-path docs/ geopic_tag_reader/
 
 help: ## Print this help message

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/PKG-INFO

@@ -1,26 +1,25 @@
 Metadata-Version: 2.1
 Name: geopic-tag-reader
-Version: 1.0.5
+Version: 1.1.0
 Summary: GeoPicTagReader
 Author-email: Adrien PAVIE <panieravide@riseup.net>
 Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 Classifier: License :: OSI Approved :: MIT License
-Requires-Dist: typer ~= 0.9.0
-Requires-Dist: xmltodict ~= 0.13.0
+Requires-Dist: typer ~= 0.12
+Requires-Dist: xmltodict ~= 0.13
 Requires-Dist: pyexiv2 == 2.8.3
+Requires-Dist: timezonefinder == 6.2.0
+Requires-Dist: pytz ~= 2023.3
+Requires-Dist: types-pytz ~= 2023.3.0.1
 Requires-Dist: flit ~= 3.8.0 ; extra == "build"
-Requires-Dist: black ~= 22.8.0 ; extra == "dev"
-Requires-Dist: mypy ~= 1.0.0 ; extra == "dev"
+Requires-Dist: black ~= 24.3 ; extra == "dev"
+Requires-Dist: mypy ~= 1.9 ; extra == "dev"
 Requires-Dist: pytest ~= 7.2.0 ; extra == "dev"
-Requires-Dist: pytest-datafiles ~= 2.0.1 ; extra == "dev"
-Requires-Dist: typer-cli-forked ~= 0.0.14 ; extra == "dev"
+Requires-Dist: pytest-datafiles ~= 3.0 ; extra == "dev"
 Requires-Dist: lazydocs ~= 0.4.8 ; extra == "dev"
-Requires-Dist: types-xmltodict ~= 0.13.0 ; extra == "dev"
+Requires-Dist: types-xmltodict ~= 0.13 ; extra == "dev"
 Requires-Dist: pre-commit ~= 3.3.3 ; extra == "dev"
-Requires-Dist: timezonefinder == 6.2.0 ; extra == "write-exif"
-Requires-Dist: pytz ~= 2023.3 ; extra == "write-exif"
-Requires-Dist: types-pytz ~= 2023.3.0.1 ; extra == "write-exif"
 Requires-Dist: python-dateutil ~= 2.8.2 ; extra == "write-exif"
 Project-URL: Home, https://gitlab.com/geovisio/geo-picture-tag-reader
 Provides-Extra: build

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/docs/reader.md

@@ -10,7 +10,7 @@
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L114"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L118"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `readPictureMetadata`
 
@@ -35,7 +35,7 @@ Extracts metadata from picture file
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L346"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L355"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeMakeModel`
 
@@ -48,7 +48,7 @@ Python 2/3 compatible decoding of make/model field.
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L357"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L366"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `isValidManyFractions`
 
@@ -63,7 +63,7 @@ isValidManyFractions(value: str) → bool
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L364"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L373"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeManyFractions`
 
@@ -76,7 +76,7 @@ Try to decode a list of fractions, separated by spaces
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L377"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L386"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeLatLon`
 
@@ -92,14 +92,16 @@ Reads GPS info from given group to get latitude/longitude as float coordinates
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L432"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L441"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeDateTimeOriginal`
 
 ```python
 decodeDateTimeOriginal(
     data: dict,
-    datetimeField: str
+    datetimeField: str,
+    lat: Optional[float] = None,
+    lon: Optional[float] = None
 ) → Tuple[Optional[datetime], List[str]]
 ```
 
@@ -110,7 +112,7 @@ decodeDateTimeOriginal(
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L464"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L495"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeTimeOffset`
 
@@ -125,12 +127,17 @@ decodeTimeOffset(data: dict, offsetTimeField: str) → Optional[tzinfo]
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L470"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L501"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeGPSDateTime`
 
 ```python
-decodeGPSDateTime(data: dict, group: str) → Tuple[Optional[datetime], List[str]]
+decodeGPSDateTime(
+    data: dict,
+    group: str,
+    lat: Optional[float] = None,
+    lon: Optional[float] = None
+) → Tuple[Optional[datetime], List[str]]
 ```
 
 
@@ -140,7 +147,7 @@ decodeGPSDateTime(data: dict, group: str) → Tuple[Optional[datetime], List[str
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L513"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L552"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `decodeSecondsAndMicroSeconds`
 
@@ -158,7 +165,7 @@ decodeSecondsAndMicroSeconds(
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L539"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L578"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>function</kbd> `isExifTagUsable`
 
@@ -185,7 +192,7 @@ Is a given EXIF tag usable (not null and not an empty string)
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L16"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L20"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>class</kbd> `CropValues`
 Cropped equirectangular pictures metadata 
@@ -226,7 +233,7 @@ __init__(
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L37"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L41"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>class</kbd> `GeoPicTags`
 Tags associated to a geolocated picture 
@@ -237,7 +244,7 @@ Tags associated to a geolocated picture
  
  - <b>`lat`</b> (float):  GPS Latitude (in WGS84) 
  - <b>`lon`</b> (float):  GPS Longitude (in WGS84) 
- - <b>`ts`</b> (float):  The capture date (as POSIX timestamp) 
+ - <b>`ts`</b> (datetime):  The capture date (date & time with timezone) 
  - <b>`heading`</b> (int):  Picture heading (in degrees, North = 0°, East = 90°, South = 180°, West = 270°) 
  - <b>`type`</b> (str):  The kind of picture (flat, equirectangular) 
  - <b>`make`</b> (str):  The camera manufacturer name 
@@ -260,7 +267,7 @@ Implementation note: this needs to be sync with the PartialGeoPicTags structure
 __init__(
     lat: float,
     lon: float,
-    ts: float,
+    ts: datetime,
     heading: Optional[int],
     type: str,
     make: Optional[str],
@@ -283,12 +290,12 @@ __init__(
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L73"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L77"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>class</kbd> `InvalidExifException`
 Exception for invalid EXIF information from image 
 
-<a href="../geopic_tag_reader/reader.py#L76"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L80"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ### <kbd>method</kbd> `__init__`
 
@@ -306,7 +313,7 @@ __init__(msg)
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L80"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L84"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>class</kbd> `PartialGeoPicTags`
 Tags associated to a geolocated picture when not all tags have been found 
@@ -321,7 +328,7 @@ Implementation note: this needs to be sync with the GeoPicTags structure
 __init__(
     lat: Optional[float] = None,
     lon: Optional[float] = None,
-    ts: Optional[float] = None,
+    ts: Optional[datetime] = None,
     heading: Optional[int] = None,
     type: Optional[str] = None,
     make: Optional[str] = None,
@@ -344,14 +351,14 @@ __init__(
 
 ---
 
-<a href="../geopic_tag_reader/reader.py#L101"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L105"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ## <kbd>class</kbd> `PartialExifException`
 Exception for partial / missing EXIF information from image 
 
 Contains a PartialGeoPicTags with all tags that have been read and the list of missing tags 
 
-<a href="../geopic_tag_reader/reader.py#L108"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
+<a href="../geopic_tag_reader/reader.py#L112"><img align="right" style="float:right;" src="https://img.shields.io/badge/-source-cccccc?style=flat-square"></a>
 
 ### <kbd>method</kbd> `__init__`
 

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/geopic_tag_reader/__init__.py

@@ -1,4 +1,5 @@
 """
 GeoPicTagReader
 """
-__version__ = "1.0.5"
+
+__version__ = "1.1.0"

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/geopic_tag_reader/main.py

@@ -22,7 +22,7 @@ def read(
 
         print("Latitude:", metadata.lat)
         print("Longitude:", metadata.lon)
-        print("Timestamp:", metadata.ts)
+        print("Timestamp:", metadata.ts.isoformat())
         print("Heading:", metadata.heading)
         print("Type:", metadata.type)
         print("Make:", metadata.make)

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/geopic_tag_reader/reader.py

@@ -7,11 +7,15 @@ import re
 import json
 from fractions import Fraction
 from geopic_tag_reader import camera
+import timezonefinder  # type: ignore
+import pytz
 
 # This is a fix for invalid MakerNotes leading to picture not read at all
 # https://github.com/LeoHsiao1/pyexiv2/issues/58
 pyexiv2.set_log_level(4)
 
+tz_finder = timezonefinder.TimezoneFinder()
+
 
 @dataclass
 class CropValues:
@@ -41,7 +45,7 @@ class GeoPicTags:
     Attributes:
         lat (float): GPS Latitude (in WGS84)
         lon (float): GPS Longitude (in WGS84)
-        ts (float): The capture date (as POSIX timestamp)
+        ts (datetime): The capture date (date & time with timezone)
         heading (int): Picture heading (in degrees, North = 0°, East = 90°, South = 180°, West = 270°)
         type (str): The kind of picture (flat, equirectangular)
         make (str): The camera manufacturer name
@@ -58,7 +62,7 @@ class GeoPicTags:
 
     lat: float
     lon: float
-    ts: float
+    ts: datetime.datetime
     heading: Optional[int]
     type: str
     make: Optional[str]
@@ -86,7 +90,7 @@ class PartialGeoPicTags:
 
     lat: Optional[float] = None
     lon: Optional[float] = None
-    ts: Optional[float] = None
+    ts: Optional[datetime.datetime] = None
     heading: Optional[int] = None
     type: Optional[str] = None
     make: Optional[str] = None
@@ -142,6 +146,11 @@ def readPictureMetadata(picture: bytes) -> GeoPicTags:
         except:
             pass
 
+    # Sanitize charset information
+    for k, v in data.items():
+        if isinstance(v, str):
+            data[k] = re.sub(r"charset=[^\s]+", "", v).strip()
+
     # Parse latitude/longitude
     lat, lon, llw = decodeLatLon(data, "Exif.GPSInfo")
     if len(llw) > 0:
@@ -163,13 +172,13 @@ def readPictureMetadata(picture: bytes) -> GeoPicTags:
         raise InvalidExifException("Read longitude is out of WGS84 bounds (should be in [-180, 180])")
 
     # Parse date/time
-    d, llw = decodeGPSDateTime(data, "Exif.GPSInfo")
+    d, llw = decodeGPSDateTime(data, "Exif.GPSInfo", lat, lon)
 
     if len(llw) > 0:
         warnings.extend(llw)
 
     if d is None:
-        d, llw = decodeGPSDateTime(data, "Xmp.exif")
+        d, llw = decodeGPSDateTime(data, "Xmp.exif", lat, lon)
         if len(llw) > 0:
             warnings.extend(llw)
 
@@ -180,7 +189,7 @@ def readPictureMetadata(picture: bytes) -> GeoPicTags:
         "Xmp.GPano.SourceImageCreateTime",
     ]:
         if d is None:
-            d, llw = decodeDateTimeOriginal(data, exifField)
+            d, llw = decodeDateTimeOriginal(data, exifField, lat, lon)
             if len(llw) > 0:
                 warnings.extend(llw)
 
@@ -313,7 +322,7 @@ def readPictureMetadata(picture: bytes) -> GeoPicTags:
             PartialGeoPicTags(
                 lat,
                 lon,
-                d.timestamp() if d else None,
+                d,
                 heading,
                 pic_type,
                 make,
@@ -330,7 +339,7 @@ def readPictureMetadata(picture: bytes) -> GeoPicTags:
     return GeoPicTags(
         lat,
         lon,
-        d.timestamp(),
+        d,
         heading,
         pic_type,
         make,
@@ -429,7 +438,9 @@ def decodeLatLon(data: dict, group: str) -> Tuple[Optional[float], Optional[floa
     return (lat, lon, warnings)
 
 
-def decodeDateTimeOriginal(data: dict, datetimeField: str) -> Tuple[Optional[datetime.datetime], List[str]]:
+def decodeDateTimeOriginal(
+    data: dict, datetimeField: str, lat: Optional[float] = None, lon: Optional[float] = None
+) -> Tuple[Optional[datetime.datetime], List[str]]:
     d = None
     warnings = []
 
@@ -442,7 +453,6 @@ def decodeDateTimeOriginal(data: dict, datetimeField: str) -> Tuple[Optional[dat
             secondsRaw, microsecondsRaw, msw = decodeSecondsAndMicroSeconds(
                 timeRaw[2], data["Exif.Photo.SubSecTimeOriginal"] if isExifTagUsable(data, "Exif.Photo.SubSecTimeOriginal", float) else "0"
             )
-            tz = decodeTimeOffset(data, f"Exif.Photo.OffsetTime{'Original' if 'DateTimeOriginal' in datetimeField else ''}")
             warnings += msw
 
             d = datetime.datetime.combine(
@@ -452,9 +462,30 @@ def decodeDateTimeOriginal(data: dict, datetimeField: str) -> Tuple[Optional[dat
                     minutesRaw,
                     secondsRaw,
                     microsecondsRaw,
-                    tzinfo=tz or datetime.timezone.utc,
                 ),
             )
+
+            # Timezone handling
+            # Try to read from EXIF
+            tz = decodeTimeOffset(data, f"Exif.Photo.OffsetTime{'Original' if 'DateTimeOriginal' in datetimeField else ''}")
+            if tz is not None:
+                d = d.replace(tzinfo=tz)
+
+            # Otherwise, try to deduct from coordinates
+            elif lon is not None and lat is not None:
+                tz_name = tz_finder.timezone_at(lng=lon, lat=lat)
+                if tz_name is not None:
+                    d = pytz.timezone(tz_name).localize(d)
+                # Otherwise, default to UTC + warning
+                else:
+                    d = d.replace(tzinfo=datetime.timezone.utc)
+                    warnings.append("Precise timezone information not found, fallback to UTC")
+
+            # Otherwise, default to UTC + warning
+            else:
+                d = d.replace(tzinfo=datetime.timezone.utc)
+                warnings.append("Precise timezone information not found (and no GPS coordinates to help), fallback to UTC")
+
         except ValueError as e:
             warnings.append("Skipping original date/time (from " + datetimeField + ") as it was not recognized:\n\t" + str(e))
 
@@ -467,7 +498,9 @@ def decodeTimeOffset(data: dict, offsetTimeField: str) -> Optional[datetime.tzin
     return None
 
 
-def decodeGPSDateTime(data: dict, group: str) -> Tuple[Optional[datetime.datetime], List[str]]:
+def decodeGPSDateTime(
+    data: dict, group: str, lat: Optional[float] = None, lon: Optional[float] = None
+) -> Tuple[Optional[datetime.datetime], List[str]]:
     d = None
     warnings = []
 
@@ -504,6 +537,12 @@ def decodeGPSDateTime(data: dict, group: str) -> Tuple[Optional[datetime.datetim
                     ),
                 )
 
+                # Set timezone from coordinates
+                if lon is not None and lat is not None:
+                    tz_name = tz_finder.timezone_at(lng=lon, lat=lat)
+                    if tz_name is not None:
+                        d = d.astimezone(pytz.timezone(tz_name))
+
         except ValueError as e:
             warnings.append(f"Skipping GPS date/time ({group} group) as it was not recognized:\n\t{str(e)}")
 

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/geopic_tag_reader/writer.py

@@ -3,11 +3,11 @@ from datetime import datetime, timedelta
 from dataclasses import dataclass
 from geopic_tag_reader.model import PictureType
 from enum import Enum
+import timezonefinder  # type: ignore
+import pytz
 
 try:
     import pyexiv2  # type: ignore
-    import timezonefinder  # type: ignore
-    import pytz
 except ImportError:
     raise Exception(
         """Impossible to write the exif tags without the '[write-exif]' dependency (that will need to install libexiv2).

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/pyproject.toml

@@ -8,9 +8,12 @@ license = {file = "LICENSE"}
 classifiers = ["License :: OSI Approved :: MIT License"]
 dynamic = ["version", "description"]
 dependencies = [
-    "typer ~= 0.9.0",
-    "xmltodict ~= 0.13.0",
+    "typer ~= 0.12",
+    "xmltodict ~= 0.13",
     "pyexiv2 == 2.8.3",
+    "timezonefinder == 6.2.0",
+    "pytz ~= 2023.3",
+    "types-pytz ~= 2023.3.0.1",
 ]
 requires-python = ">=3.8"
 
@@ -22,22 +25,17 @@ geopic-tag-reader="geopic_tag_reader.main:app"
 
 [project.optional-dependencies]
 dev = [
-    "black ~= 22.8.0",
-    "mypy ~= 1.0.0",
+    "black ~= 24.3",
+    "mypy ~= 1.9",
     "pytest ~= 7.2.0",
-    "pytest-datafiles ~= 2.0.1",
-    "typer-cli-forked ~= 0.0.14", # fork dependency needed until https://github.com/tiangolo/typer-cli/pull/120 is merged.
-    #"typer-cli ~= 0.0.13",
+    "pytest-datafiles ~= 3.0",
     "lazydocs ~= 0.4.8",
-    "types-xmltodict ~= 0.13.0",
+    "types-xmltodict ~= 0.13",
     "pre-commit ~= 3.3.3",
 ]
 build = ["flit ~= 3.8.0"]
 # optional dependencies to be able to write exif tags
 write-exif = [
-    "timezonefinder == 6.2.0",
-    "pytz ~= 2023.3",
-    "types-pytz ~= 2023.3.0.1",
     "python-dateutil ~= 2.8.2",
 ]
 

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/tests/test_reader.py

@@ -7,7 +7,6 @@ from .conftest import FIXTURE_DIR, openImg
 def assertGeoPicTagsEquals(gpt, expectedDict):
     assert gpt.lat == expectedDict.get("lat")
     assert gpt.lon == expectedDict.get("lon")
-    assert gpt.ts == expectedDict.get("ts")
     assert gpt.heading == expectedDict.get("heading")
     assert gpt.type == expectedDict.get("type")
     assert gpt.make == expectedDict.get("make")
@@ -17,6 +16,12 @@ def assertGeoPicTagsEquals(gpt, expectedDict):
     assert gpt.tagreader_warnings == expectedDict.get("tagreader_warnings", [])
     assert len(gpt.exif) > 0
 
+    if expectedDict.get("ts") is not None:
+        assert gpt.ts is not None
+        assert gpt.ts.isoformat() == expectedDict["ts"]
+    else:
+        assert gpt.ts is None
+
     if gpt.crop:
         assert expectedDict.get("crop") is not None
         assert gpt.crop.fullWidth == expectedDict["crop"].get("fullWidth")
@@ -37,7 +42,7 @@ def test_readPictureMetadata(datafiles):
         {
             "lat": 49.00688961988304,
             "lon": 1.9191854417991367,
-            "ts": 1627550214.0,
+            "ts": "2021-07-29T11:16:54+02:00",
             "heading": 349,
             "type": "equirectangular",
             "make": "GoPro",
@@ -56,7 +61,7 @@ def test_readPictureMetadata_negCoords(datafiles):
         {
             "lat": 48.33756428166505,
             "lon": -1.9331088333333333,
-            "ts": 1652453580.0,
+            "ts": "2022-05-13T16:53:00+02:00",
             "heading": 32,
             "type": "equirectangular",
             "make": "GoPro",
@@ -75,7 +80,7 @@ def test_readPictureMetadata_flat(datafiles):
         {
             "lat": 48.139852239480945,
             "lon": -1.9499731060073981,
-            "ts": 1429976268.0,
+            "ts": "2015-04-25T15:37:48+02:00",
             "heading": 155,
             "type": "flat",
             "make": "OLYMPUS IMAGING CORP.",
@@ -93,7 +98,7 @@ def test_readPictureMetadata_flat2(datafiles):
         {
             "lat": 48.85779642035038,
             "lon": 2.3392783047650747,
-            "ts": 1430744932.0,
+            "ts": "2015-05-04T13:08:52+02:00",
             "heading": 302,
             "type": "flat",
             "make": "Canon",
@@ -111,7 +116,7 @@ def test_readPictureMetadata_xmpHeading(datafiles):
         {
             "lat": 50.87070833333333,
             "lon": -1.5260916666666666,
-            "ts": 1600008019.767,
+            "ts": "2020-09-13T15:40:19.767000+01:00",
             "heading": 67,
             "type": "equirectangular",
             "make": "Google",
@@ -130,7 +135,7 @@ def test_readPictureMetadata_noHeading(datafiles):
         {
             "lat": 48.15506638888889,
             "lon": -1.6844680555555556,
-            "ts": 1666166194.0,
+            "ts": "2022-10-19T09:56:34+02:00",
             "heading": None,
             "type": "flat",
             "make": "SONY",
@@ -143,7 +148,7 @@ def test_readPictureMetadata_noHeading(datafiles):
 
 @pytest.mark.datafiles(os.path.join(FIXTURE_DIR, "img_Ricoh_Theta.jpg"))
 def test_readPictureMetadata_ricoh_theta(datafiles):
-    for f in datafiles.listdir():
+    for f in datafiles.iterdir():
         result = reader.readPictureMetadata(openImg(str(f)))
         assertGeoPicTagsEquals(
             result,
@@ -154,7 +159,7 @@ def test_readPictureMetadata_ricoh_theta(datafiles):
                 "lon": 2.3205357914890987,
                 "make": "RICOH",
                 "model": "THETA m15",
-                "ts": 1458911533.0,
+                "ts": "2016-03-25T14:12:13+01:00",
                 "type": "equirectangular",
             },
         )
@@ -162,7 +167,7 @@ def test_readPictureMetadata_ricoh_theta(datafiles):
 
 @pytest.mark.datafiles(os.path.join(FIXTURE_DIR, "img_V4MPack.jpg"))
 def test_readPictureMetadata_v4mpack(datafiles):
-    for f in datafiles.listdir():
+    for f in datafiles.iterdir():
         result = reader.readPictureMetadata(openImg(str(f)))
         assertGeoPicTagsEquals(
             result,
@@ -173,7 +178,7 @@ def test_readPictureMetadata_v4mpack(datafiles):
                 "lon": -1.2761512389983616,
                 "make": "STFMANI",
                 "model": "V4MPOD 1",
-                "ts": 1555417213.0,
+                "ts": "2019-04-16T14:20:13+02:00",
                 "type": "equirectangular",
                 "altitude": 34,
             },
@@ -192,7 +197,7 @@ def test_readPictureMetadata_a5000(datafiles):
             "lon": 2.51197323068765,
             "make": "OnePlus",
             "model": "ONEPLUS A5000",
-            "ts": 1626797632.199995,
+            "ts": "2021-07-20T16:13:52.199995+02:00",
             "type": "flat",
             "altitude": 0,
         },
@@ -227,7 +232,7 @@ def test_readPictureMetadata_int_long_tag(datafiles):
             "lon": 4.700622,
             "make": None,
             "model": None,
-            "ts": 1673515020.0,
+            "ts": "2023-01-12T09:17:00+01:00",
             "type": "flat",
             "tagreader_warnings": ["GPSLongitudeRef not found, assuming GPSLongitudeRef is East"],
         },
@@ -249,7 +254,7 @@ def test_readPictureMetadata_invalidGpsDate(datafiles):
             "lon": 2.358506155368721,
             "make": "samsung",
             "model": None,
-            "ts": 1631101437.0754,
+            "ts": "2021-09-08T11:43:57.075400+02:00",
             "type": "flat",
             "altitude": 73,
             "tagreader_warnings": [
@@ -271,7 +276,7 @@ def test_readPictureMetadata_gpsDateStamp(datafiles):
             "lon": 2.4833194444444446,
             "make": "Apple",
             "model": "iPhone 12 Pro",
-            "ts": 1682785851.565,
+            "ts": "2023-04-29T18:30:51.565000+02:00",
             "type": "flat",
             "altitude": 36,
             "tagreader_warnings": [
@@ -293,7 +298,7 @@ def test_readPictureMetadata_gpsDateString(datafiles):
             "lon": -1.0015555555555555,
             "make": "Apple",
             "model": "iPhone 8 Plus",
-            "ts": 1564313142.529,
+            "ts": "2019-07-28T13:25:42.529000+02:00",
             "type": "flat",
             "altitude": 19,
             "tagreader_warnings": [
@@ -322,7 +327,7 @@ def test_readPictureMetadata_cropped(datafiles):
             "lon": 1.919185441804927,
             "make": "GoPro",
             "model": "Max",
-            "ts": 1627550214.0,
+            "ts": "2021-07-29T11:16:54+02:00",
             "type": "equirectangular",
             "altitude": 93,
             "crop": {"fullWidth": 4032, "fullHeight": 2016, "width": 2150, "height": 1412, "top": 134, "left": 538},
@@ -341,7 +346,7 @@ def test_readPictureMetadata_datetimeoriginal(datafiles):
             "lon": -1.382302762883527,
             "make": "Motorola",
             "model": "XT1052",
-            "ts": 1598866588.0,
+            "ts": "2020-08-31T09:36:28+02:00",
             "type": "flat",
             "altitude": 72,
             "tagreader_warnings": [
@@ -362,7 +367,7 @@ def test_readPictureMetadata_datetimeoriginal_decimal_milliseconds(datafiles):
             "lon": 121.15072633333334,
             "make": "BlackVue",
             "model": "DR900S-1CH",
-            "ts": 1699748719.023,
+            "ts": "2023-11-12T00:25:19.023000+08:00",
             "type": "flat",
             "altitude": 29,
             "tagreader_warnings": [],
@@ -401,7 +406,7 @@ def test_readPictureWithoutCoord(datafiles):
             "lon": None,
             "make": "Apple",
             "model": "iPhone 12 Pro",
-            "ts": 1682785851.565,
+            "ts": "2023-04-29T16:30:51.565000+00:00",
             "type": "flat",
             "altitude": 36,
             "tagreader_warnings": [
@@ -472,7 +477,7 @@ def test_readPictureWithLatitudeAsFloat(datafiles):
             "lon": 7.683081,
             "make": None,
             "model": "PULSAR",
-            "ts": 1637075896.89,
+            "ts": "2021-11-16T16:18:16.890000+01:00",
             "type": "equirectangular",
             "altitude": None,
             "tagreader_warnings": [],
@@ -492,7 +497,7 @@ def test_readPictureMetadata_gps_date_time_stamp(datafiles):
             "lon": 7.683081,
             "make": None,
             "model": "PULSAR",
-            "ts": 1637075896.89,
+            "ts": "2021-11-16T16:18:16.890000+01:00",
             "type": "equirectangular",
             "altitude": None,
             "tagreader_warnings": [],
@@ -512,7 +517,7 @@ def test_readPictureMetadata_insta360(datafiles):
             "lon": 3.482755555555556,
             "make": "Insta360",
             "model": "One X2.PHOTO_NORMAL",
-            "ts": 1700662669.0,
+            "ts": "2023-11-22T14:17:49+01:00",
             "type": "equirectangular",
             "altitude": 84,
             "tagreader_warnings": [],
@@ -532,7 +537,7 @@ def test_readPictureMetadata_ricoh_noproj(datafiles):
             "lon": -1.5464416503906249,
             "make": "Ricoh",
             "model": "Theta S",
-            "ts": 1672575342.0,
+            "ts": "2023-01-01T13:15:42+01:00",
             "type": "equirectangular",
             "altitude": None,
             "tagreader_warnings": [],
@@ -549,7 +554,7 @@ def test_readPictureMetadata_gopromax_flat(datafiles):
         {
             "lat": 47.22555109997222,
             "lon": -1.5631604999722222,
-            "ts": 1708173992.0,
+            "ts": "2024-02-17T13:46:32+01:00",
             "heading": None,
             "type": "flat",
             "make": "GoPro",
@@ -572,7 +577,7 @@ def test_readPictureMetadata_broken_makernotes(datafiles):
             "lon": -1.7841980555555554,
             "make": "SONY",
             "model": "FDR-X1000V",
-            "ts": 1599115820.0,
+            "ts": "2020-09-03T08:50:20+02:00",
             "type": "flat",
             "altitude": 99,
             "focal_length": 2.8,
@@ -590,7 +595,7 @@ def test_readPictureMetadata_datetime_offset(datafiles):
         {
             "lat": 48.33756428166505,
             "lon": -1.9331088333333333,
-            "ts": 1652453651.0,
+            "ts": "2022-05-13T16:54:11+02:00",
             "heading": 32,
             "type": "equirectangular",
             "make": "GoPro",
@@ -610,7 +615,7 @@ def test_readPictureMetadata_gps_date_slash(datafiles):
         {
             "lat": 43.42541569992266,
             "lon": 1.3766216000638112,
-            "ts": 1686640500.0,
+            "ts": "2023-06-13T09:15:00+02:00",
             "heading": None,
             "type": "flat",
             "make": None,
@@ -623,3 +628,9 @@ def test_readPictureMetadata_gps_date_slash(datafiles):
             ],
         },
     )
+
+
+@pytest.mark.datafiles(os.path.join(FIXTURE_DIR, "charset.jpg"))
+def test_readPictureMetadata_charset(datafiles):
+    result = reader.readPictureMetadata(openImg(str(datafiles) + "/charset.jpg"))
+    assert result.exif["Exif.Photo.UserComment"] == "CD31 31_D0003_51_600"

{geopic_tag_reader-1.0.5 → geopic_tag_reader-1.1.0}/tests/test_writer.py

@@ -18,7 +18,7 @@ def test_writePictureMetadata_capture_time(datafiles):
     image_file_upd = writer.writePictureMetadata(img_orig, writer.PictureMetadata(capture_time=capture_time))
     tags = reader.readPictureMetadata(image_file_upd)
 
-    assert datetime.fromtimestamp(tags.ts, tz=pytz.UTC) == capture_time
+    assert tags.ts == capture_time
 
     # we also check specific tags:
     assert tags.exif["Exif.Photo.DateTimeOriginal"] == "2023-06-01 12:48:01"
@@ -36,7 +36,7 @@ def test_writePictureMetadata_capture_time_no_timezone(datafiles):
     tags = reader.readPictureMetadata(image_file_upd)
 
     paris = pytz.timezone("Europe/Paris")
-    assert datetime.fromtimestamp(tags.ts, tz=pytz.UTC) == paris.localize(capture_time).astimezone(pytz.UTC)
+    assert tags.ts == paris.localize(capture_time).astimezone(pytz.UTC)
 
     # DateTimeOriginal should be a local time, so 12:48:01 localized in Europe/Paris timezome (since it's where the picture has been taken)
     assert tags.exif["Exif.Photo.DateTimeOriginal"] == "2023-06-01 12:48:01"
@@ -78,7 +78,7 @@ def test_writePictureMetadata_capture_time_no_position_in_file_but_overriden(dat
     tags = reader.readPictureMetadata(image_file_upd)
 
     paris = pytz.timezone("Europe/Paris")
-    assert datetime.fromtimestamp(tags.ts, tz=pytz.UTC) == paris.localize(capture_time).astimezone(pytz.UTC)
+    assert tags.ts == paris.localize(capture_time).astimezone(pytz.UTC)
 
     # DateTimeOriginal should be a local time, so 12:48:01 localized in Europe/Paris timezome (since it's where the picture has been taken)
     assert tags.exif["Exif.Photo.DateTimeOriginal"] == "2023-06-01 12:48:01"