Skip to content

Handle malformed video GPX sidecars - #845

Open
rtwojcik wants to merge 1 commit into
mainfrom
fix/malformed-video-gpx-sidecar
Open

rtwojcik wants to merge 1 commit into
mainfrom
fix/malformed-video-gpx-sidecar

Conversation

@rtwojcik

@rtwojcik rtwojcik commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Report malformed automatically discovered video GPX sidecars as a typed
    processing error instead of aborting processing without a usable description.
  • Keep the error associated with the video so clients can retain it and offer a
    replacement workflow.
  • Use a platform-agnostic, basename-only error message.

Test plan

  • pytest -q tests — 727 passed, 52 skipped
  • ruff check mapillary_tools
  • ruff format --check mapillary_tools tests
  • usort diff mapillary_tools
  • mypy mapillary_tools tests/cli

The focused integration test exercises the command through its normal
multiprocessing boundary and verifies the serialized error type and message.

@meta-cla meta-cla Bot added the cla signed label Oct 8, 2026
@rtwojcik
rtwojcik requested a review from caglarpir October 8, 2026 11:12
@rtwojcik
rtwojcik marked this pull request as ready for review October 8, 2026 12:58
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.

1 participant