Skip to content

Conversation

ptpt
Copy link
Member

@ptpt ptpt commented Apr 1, 2025

Process with native parsing first, then ExifTool (the default behaviour)

When running process, MT now defaults to using native parsers (exif, camm, gopro, blackvue) before falling back to ExifTool for any failures. This approach leverages our previous speedup work (#722, #729, #731, #721) making native parsing 3-10x faster than ExifTool, while maintaining ExifTool's broad format support.

If ExifTool is not found, a warning is printed:

2025-03-31 20:08:05,510 - WARNING - Skip "exiftool_runtime" because: [Errno 2] No such file or directory: 'exiftool2'

You can specify the path to your ExifTool installation with the MAPILLARY_TOOLS_EXIFTOOL_PATH environment variable. When not set, MT searches for ExifTool in your PATH.

Unified Geotag Source Options

Reused the same geotag option syntax here but standardized the support for both videos and images:

  • --geotag_source=exif expands to {"source": "exif"} (applies to images only)
  • --geotag_source=native,exiftool_runtime expands to [{"source": "native"}, {"source": "exiftool_runtime"}]

The --video_geotag_source option is now a convenience wrapper for --geotag_source that always targets videos:

  • --video_geotag_source=exiftool_runtime expands to {"source": "exiftool_runtime", "filetypes": "video"}

@ptpt ptpt merged commit 7a45e5e into main Apr 1, 2025
19 checks passed
@ptpt ptpt deleted the improve-unify-geotag-sources branch April 1, 2025 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants