Set priority when selecting location from nominatim #277
Labels
Milestone
Comments
|
Ah, would this be better reported on Project-OSRM-Web then? |
|
It is perfectly fine to report it here. Am 26.05.2012 um 10:48 schrieb Raphaël Pinson reply@reply.github.com:
|
|
The easiest workaround is to reorder the result from Nomination in OSRM. |
|
The following PR should fix this issue: Project-OSRM/osrm-frontend-legacy#6 |
|
After talking with @twain47, it appears that these functionalities are already implemented in Nominatim:
|
springmeyer
pushed a commit
that referenced
this issue
Dec 15, 2016
76de7eb bump to v0.3.0 a2f36f0 Merge pull request #301 from mapbox/v8-5.1.281.47 e06010e Merge pull request #300 from mapbox/v8-3.14.5.10 61021d9 build xcode-toolchain for darwin [skip ci] a17d47e Update Android compile flags 8adc05a Merge pull request #298 from mapbox/c11-for-tc 91cc1ce adjustments to tippecanoe build 3693450 download tar of 1.15.1 1563d66 update to c++11 for tippecanoe ca2cf2f fix conditional building of libcxx with llvm 4.x e92a050 Add iwyu to llvm packages + conditionally build libc++ (to avoid potential conflicts with apple system) ecbe161 clang++ subpackage: rebundling asan_symbolize from llvm a04d31a llvm: install asan_symbolize script f81ff0f Merge pull request #297 from mapbox/boost_fixes 3c4201f also trigger build for header-only package 394db3d only need to build the header-only package once for all platforms 6f6dd89 avoid the header-only package reporting ldflags 2bef25b fix boost library name for ldflags 8abd030 bump to v0.2.0 434da41 Merge pull request #293 from mapbox/gdb 2c5e104 add texinfo [skip ci] 001e1d0 build gdb with g++ [skip ci] fa89a91 attempt to workaround gdb bug [skip ci] 7a573b0 add gdb [skip ci] 33b8c85 Merge pull request #292 from mapbox/binutils-latest d88af66 Upgrade Android NDK to r13b ce29128 add Android NDK r13b [skip ci] fcaff69 include llvm-ar and llvm-ranlib in clang++ package for full LTO support on linux 5fa738d add disable-werror to binutils build [skip ci] b38b22e add texinfo dep [skip ci] a9c038a add latest binutils 09a1272 add geojson 0.4.0 [skip ci] f787753 add geometry 0.9.0 [skip ci] 649178c add variant 1.1.4 [skip ci] f29bc15 Merge pull request #285 from mapbox/pkgconfig 5c3a26d add pkgconfig 0.29.1 228784d Fix bug in wagyu e9cfd69 Merge pull request #284 from mapbox/wagyu2 0dab0ba Update travis script b580a12 Updated hash as it was not md5 -- something I forgot a5c6b7b Merge pull request #283 from mapbox/wagyu dd05da8 Added wagyu 0.1.0 5407ddc Merge pull request #282 from mapbox/boost-1.62.0 fa6d51a add boost 1.62.0, remove obsolete boost packages d0210bb drop the boost 'all' packages 66d8a26 remove empty before_install 7527555 Merge pull request #281 from mapbox/mason-travis-token af4a2c6 [gtest@1.7.0] Skip linking with pthread on android 2115889 rename TRAVIS_TOKEN to MASON_TRAVIS_TOKEN to avoid conflicting with travis.rb (https://github.com/travis-ci/travis.rb/blob/b75c38bcdb736d1011e0d2a0022344a9870acaf4/README.md#environment-variables) 8117c1c Merge pull request #279 from mapbox/llvm-libstdcxx 3c9029e strip version modifier 9e1edca Add llvm 3.8.1 package that sets BUILD_AND_LINK_LIBCXX=false - refs #278 9724ef5 make building and linking libcxx/libcxxabi optional - refs #278 6c4c5d5 solve #152 8f3a877 Merge pull request #277 from mapbox/central-secure 9350c34 New travis keys f442ddc add secure variables on-demand in trigger command 36d4801 remove global variables 4fff08a Add mesa 13.0.0-egl{,-cxx11abi} e3f2a2c [mason] Add support for nested pkg-config files 3b79550 Add ICU 58.1 (#275) 20a394d make contributing.md tagging more DRY [skip ci] 58ae447 bump to v0.1.1 c5ef33f rename llvm 4.x to 4.0.0 [skip ci] 5d5d26b add llvm 4.x based sub-packages 9708680 only set LIBCXX_ENABLE_STATIC_ABI_LIBRARY for linux [skip ci] 6918fb0 install LLVMgold.so on linux [skip ci] ab20ff7 binutils: using 'all-gold' breaks make install on linux [skip ci] cb8b15f fix binutils install [skip ci] df77b02 properly set LLVM_BINUTILS_INCDIR value [skip ci] 295d4db binutils gold fixes [skip ci] ac85676 enable LLVMgold.so for linux [skip ci] acc4249 add binutils 2.27 16d4452 add lldb and llvm-cov 3.8.1 packages 757b908 remove patching no longer needed [skip ci] 69415b5 better fix for statically linking/linked libc++ with llvm 3.8.1 [skip ci] 43933ed Added mesa-13.0.0-glx (non-CXX11ABI) 2bb81cc Added mesa-13.0.0-glx d70e3ef LLVM_EXTERNALIZE_DEBUGINFO is osx specific ba737bf llvm 3.8.1 linux build fixes 4279f58 Updates for mesa 13.0.0 d544278 Revert typo in mason.sh 03ecb2c Add mesa 13.0.0 (Gallium-OSMesa) 0997905 Add expat 2.2.0 211d8f4 v8 libplatform depends on libbase [skip ci] c9d2371 thin archives are only an linux thing [skip ci] bd4964b ensure v8 static libs are portable [skip ci] f41eb6d mason.cmake: multiple static libs also need to be split c9fcd1a add lldb to llvm 3.8.1 [skip ci] 6d8e16c minor llvm/clang++ improvements 65a5418 fix static_libs 31a424f add v8 5.1.281.47 1afa241 fix v8 include dir [skip ci] b2bdb33 Add note that we need to create a github release [skip ci] 17d9b09 linux fixes 95b4f2e add v8 3.14.5.10 git-subtree-dir: third_party/mason git-subtree-split: cf3561bf78cb146821dba3d95c308fc44db05f46
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As mentionned on Twitter, when Nominatim returns more than one result for a location, it is common that the default value be an administrative boundary, which is usually located in very weird places (on the outside of the boundary).
It would be best to sort the Nominatim results, giving preference to "place" and "amenity" classes before others, so as to select a location that makes sense when people search for, e.g. a city name.
See for example a search on Chambery which returns this json, where the best point to pick would be ID 1522668290 (Chambéry, Savoie, Rhône-Alpes, 73000, France), which is a place.
The text was updated successfully, but these errors were encountered: