GIScience / GIScience/openrouteservice

Improve snapping radius implementation

Open
#985 0 comments 0 reactions 0 assignees View on GitHub
demanding 💪 investigate
Dominant language
Java
Stars
2k
Forks
481
Avg merge
1d 10h
Merged PRs (30d)
59

Description

In the current implementation of snapping radius the limit is applied on top of the result returned by the `findClosest` method of `LocationIndex` class. Instead, investigate options to pass the radius directly to the lookup query in order to avoid searching an unnecessarily large area, and to be able to expand the search radius on a per-query basis (basically, override the `location_index_search_iterations` config file setting).

A potential starting point to look into could be the `LocationIndexTree.findNClosest` method.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.