GIScience / GIScience/openrouteservice

Rename inconsistently named parameters in the configuration

Open
#554 1 comment 1 reaction 0 assignees View on GitHub
cleanup :broom: configuration easy-fix 😎
Dominant language
Java
Stars
2k
Forks
481
Avg merge
1d 10h
Merged PRs (30d)
59

Description

Both app.config parameters `maximum_snapping_radius`(routing) and `maximum_search_radius`(matrix) serve the purpose to define the maximum distance from the graph a given coordinate may have in order to find a nearest point on the graph. Serving the same purpose, they should have the same name. `maximum_snapping_radius` is to be preferred as it states more clearly what the parameter is about.

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.