-
Notifications
You must be signed in to change notification settings - Fork 460
Open
Labels
Description
Is there an existing issue for this?
- I have searched the existing issues
Problem description
The code for snapping locations to the graph is used at several places and more or less copy-pased in several variants. It is desirable to de-duplicate this for sake of maintainability.
Proposed solution
The experimental PR #2192 contains a commit show-casing how such a refactoring could look like: 4fe7b15
Proposed steps:
- introduce class similar to Snapper as in said commit
- refactor snapping end-point to use the new Snapper class
- refactor map-matching endpoint to use the new Snapper class
- refactor MatrixSearchContextBuilder to use the new Snapper class
- optional: properly introduce external point-data-functionality provided by said PR
Additional context
No response
Forum Topic Link
No response
aoles