Skip to content

refactor snapping code #2193

@sfendrich

Description

@sfendrich

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

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions