-
Notifications
You must be signed in to change notification settings - Fork 406
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
ORS does not work with OSM from Geofabrik #839
Comments
I can confirm this issue. Neither can http://download.openstreetmap.fr/extracts/europe/austria.osm.pbf be read. Not sure where to go from here, so I cant really use ORS at all :-/ |
Not a ORS issue, it's a read timeout on a network request when downloading elevation, see line here is the fix: https://ask.openrouteservice.org/t/timeout-exception-on-elevation-data/2345/2 |
I have mounted conf folder and edit |
Ok, then two things:
Sorry, on mobile right now.. If you still can’t build then change the elevation provider in the config to gmted, that should work. |
I am not sure why you point to the tomcat directory, in documentation it says that config is at
|
No relation to Geofabrik here, it's simply the CGIAR download page being down since ages apparently. That's a recurring problem, we encountered it multiple times already.. And CGIAR keeps shifting their download URLs.. Probably time for
Coming back to the problem:
True, but if you look into the docker-entrypoint script you'll see that it'll be copied to the tomcat folder.
Right, there's indeed nothing in the default config.. I'm pretty sure though that there's supposed to be a Anyways, set |
Can you explain how to set this parameter with an example i would appreciate it very much. |
You need to set the value of the JSON path
BTW, if anyone would bother to address this open issue that wouldn't even be a problem and close at least #845 and this one: |
I have created PR #850, but I am not sure if this fixes it. I have tested it on my side and got mixed results, @nilsnolde can you verify, please? |
I'm not really part of this project anymore, but I can at least tell you that this, while nice of you to offer a PR, is not really what should be done. You can do that to your local copy of the app.config, but changing the defaults for everyone would be curing the symptoms not the cause:) and gmted is far inferior to CGIAR, so best is to have it back here: The issue I pointed to in that GH fork is the one which needs fixing. You can look at this PR where I did the exact same a few years back (this keeps happening with CGIAR..): GIScience/graphhopper#25 I guess the GH fork's branch to PR to is this one: https://github.com/GIScience/graphhopper/tree/ors_0.13 But might want to verify with the people who should actually answer issues like these;) |
Alright, thanks for the response. Waiting until further comments in this thread. Please, note that until that point the current version simply does not work under any scenario as it does not start at all. |
Jep, absolutely, should have the highest priority.. I PRd: GIScience/graphhopper#35. If you know how you can try to test. Failed for me for some reason, but really shouldn't.. |
@neatnerd , @nilsnolde sorry for the wait.. this should be fixed in 6.3.3 thanks to your pull request. |
FYI, if the latest patch 6.3.3 doesn't work, check to make sure you rebuild the image with I was getting |
Here's what I did
Here's what I got
Here's what I was expecting
I followed the instruction and expected the application to work for the selected osm file
Here's what I think could be improved
I believe that issue is related to the version of the graphhopper used. The project uses
v0.13.9
, while the latest version is ca. 2.The text was updated successfully, but these errors were encountered: