Skip to content

Add configurable and inferred base coords for address resolution#72

Merged
eifinger merged 1 commit intomainfrom
base_coords
Mar 13, 2026
Merged

Add configurable and inferred base coords for address resolution#72
eifinger merged 1 commit intomainfrom
base_coords

Conversation

@eifinger
Copy link
Owner

@eifinger eifinger commented Mar 6, 2026

Summary

  • add optional base_coords parameter to calc_routes() to control Waze address resolution anchoring
  • pass optional base coordinates through to address_to_coords()
  • when base_coords is not provided and exactly one endpoint is already coordinates, use that endpoint as the base for resolving the address
  • add tests for explicit custom base coordinates and automatic coordinate-endpoint fallback
  • document the new behavior in README.md

Validation

  • uv run pytest -q

Attribution

Inspired by @SylvainM98

@eifinger eifinger changed the title Support custom and inferred base coordinates for address resolution Add configurable and inferred base coords for address resolution Mar 13, 2026
@eifinger eifinger added the enhancement New feature or request label Mar 13, 2026
@eifinger eifinger merged commit 1d4dbe7 into main Mar 13, 2026
4 checks passed
@eifinger eifinger deleted the base_coords branch March 13, 2026 06:54
@SylvainM98
Copy link

Thanks for the credit, glad it helped ! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants