[Marble-devel] Review Request: Perform local searches
Thibaut Gridel
tgridel at free.fr
Sat Jul 28 16:34:32 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105772/
-----------------------------------------------------------
Review request for Marble.
Description
-------
I collapsed many commits together so that it forms only one review, so I'll describe more:
- first part changes the api for SearchRunner, adding a LatLonAltBox parameter. This impacts all search runners in consequence, but not all will provide the functionality.
- nominatim-search adds the &viewbox= parameters to uri, performing a restrained search
- local-osm-search takes into account the center of that bbox and sorts the results by proximity.
- localdatabase filters the results based on that bbox.
MarbleRunnerManager needs to change its caching such that asking the same searchTerm on another bbox triggers a new search.
Last, the NavigationWidget enables that Local search through a checkbox, as I couldn't find easy way to decide if a search could be local or not from the searchTerm only.
Ideas for future:
Perform local search by default might be the way to go, and asking for a general search a second class citizen.
local-osm-search has quite advanced analysis of search, maybe bringing this to main would benefit others.
Sorting by distance or by relevance is undecided as well.
Diffs
-----
src/lib/MarbleAbstractRunner.h f3d9665
src/lib/MarbleAbstractRunner.cpp 9e14e37
src/lib/MarbleRunnerManager.h 88153df
src/lib/MarbleRunnerManager.cpp 8e0589d
src/lib/NavigationWidget.h 1b509f5
src/lib/NavigationWidget.cpp 706daa1
src/lib/NavigationWidget.ui 4c259a6
src/lib/RunnerTask.h eafd719
src/lib/RunnerTask.cpp 2bb020a
src/plugins/runner/hostip/HostipRunner.h ca2c745
src/plugins/runner/hostip/HostipRunner.cpp 6b161ba
src/plugins/runner/latlon/LatLonRunner.h a7b16bf
src/plugins/runner/latlon/LatLonRunner.cpp 73b23b7
src/plugins/runner/local-osm-search/DatabaseQuery.h ec00d97
src/plugins/runner/local-osm-search/DatabaseQuery.cpp 87c6b0f
src/plugins/runner/local-osm-search/LocalOsmSearchRunner.h 1795494
src/plugins/runner/local-osm-search/LocalOsmSearchRunner.cpp 9a82e80
src/plugins/runner/local-osm-search/OsmDatabase.h 10c7beb
src/plugins/runner/local-osm-search/OsmDatabase.cpp 477708a
src/plugins/runner/localdatabase/LocalDatabaseRunner.h 034919b
src/plugins/runner/localdatabase/LocalDatabaseRunner.cpp 7d469bb
src/plugins/runner/nominatim-search/OsmNominatimSearchRunner.h d8337a8
src/plugins/runner/nominatim-search/OsmNominatimSearchRunner.cpp 1345dfa
Diff: http://git.reviewboard.kde.org/r/105772/diff/
Testing
-------
Searching by poi terms now brings interesting results.
Unchecking still allows to search for the numerous city homonyms, and specific adress still works in both cases.
Other search usecases unaffected: routing still performs unboxed search.
Thanks,
Thibaut Gridel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20120728/86d01e9e/attachment.html>
More information about the Marble-devel
mailing list