-
Notifications
You must be signed in to change notification settings - Fork 386
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add snapping endpoint #1524
Conversation
ors-api/src/main/java/org/heigit/ors/api/responses/snapping/SnappingResponse.java
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/responses/snapping/json/JsonSnappingResponse.java
Show resolved
Hide resolved
ors-api/src/test/java/org/heigit/ors/apitests/snapping/ParamsTest.java
Outdated
Show resolved
Hide resolved
ors-engine/src/main/java/org/heigit/ors/snapping/SnappingRequest.java
Outdated
Show resolved
Hide resolved
I can't run the normal Application run configuration of spring-boot.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One general question we should consider is wether we want to return this as geojson as well?
apart from that some smaller issues to resolve.
ors-api/src/main/java/org/heigit/ors/api/responses/snapping/SnappingResponse.java
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/controllers/SnappingAPI.java
Outdated
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/controllers/SnappingAPI.java
Outdated
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/controllers/SnappingAPI.java
Outdated
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/controllers/SnappingAPI.java
Outdated
Show resolved
Hide resolved
ors-api/src/main/java/org/heigit/ors/api/responses/snapping/json/JsonSnappingResponse.java
Show resolved
Hide resolved
ors-api/src/test/java/org/heigit/ors/apitests/snapping/ParamsTest.java
Outdated
Show resolved
Hide resolved
@MichaelsJP @sfendrich Do we want this endpoint on the live API or should it be hidden in the Playground? |
I'd say we open it up on the live API with the next major Release 8. |
@TheGreatRefrigerator Should be effortlessly integrable and would make the endpoint similar to the others. I'd say let's do it. |
So, just three things missing:
Adjust the documentation? |
|
0caacdc
to
58a093c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Add endpoint to
services
listed at localhost:8082/ors/v2/status. - Instead of the somewhat lengthy API parameter name
maximum_search_radius
wouldn't justradius
be sufficient?
f1d971d
to
24669a7
Compare
Snap is added to Status |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following TODO needs to be addressed.
openrouteservice/ors-engine/src/main/java/org/heigit/ors/snapping/SnappingRequest.java
Line 38 in b392a0c
// TODO: replace usage of matrix search context by snapping-specific class |
ors-engine/src/main/java/org/heigit/ors/snapping/SnappingRequest.java
Outdated
Show resolved
Hide resolved
cb125ca
to
20e2337
Compare
f74ae36
to
cb125ca
Compare
18b902b
to
d0bdc81
Compare
Co-authored-by: Jochen Haeussler <jochen.haeussler@heigit.org>
- use argument stream for parametrization - add multiple test cases
- add more tests with missing endpoint/profile
- add schema classes for GeoJSON response - add endpoint to SnappingAPI - add GeoJSON to APIEnums
ff75264
to
efb7f76
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks ready now
changes implemented review re-requested
Pull Request Checklist
have been resolved.
[Unreleased] heading.
along with a short description of what it is for, and documented this in the Pull Request (below).
(at least Germany), and the graphs build without problems (i.e. no out-of-memory errors).
importer etc.), I have generated longer distance routes for the affected profiles with different options
(avoid features, max weight etc.) and compared these with the routes of the same parameters and start/end
points generated from the current live ORS.
If there are differences then the reasoning for these MUST be documented in the pull request.
and why the change was needed.
Fixes #1519 .
Information about the changes