pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Julian Simioni	b188a2046c	Fix mismatching name of types preference parameter The parameter in clean.types being set by `helpers/text_parser.js` was "from_address_parsing", but the code in `helper/types.js` was expecting "from_address_parser". This commit makes both use "from_address_parser" and adds a test.	9 years ago
Julian Simioni	59b70f7c7e	Rename helper/query_parser to helper/text_parser	9 years ago
Julian Simioni	8f1c8f4d89	Add helpful comments in various query parser locations	9 years ago
Stephen Hess	d732021d5e	fixed tests	9 years ago
Stephen K Hess	7fb0112224	Merge pull request #304 from pelias/288-viewport-sanitiser focus.viewport sanitiser	9 years ago
Julian Simioni	868b59554a	Merge branch 'fix-place-types'	9 years ago
Julian Simioni	0e8f4253c0	Return multiple results in place when osm node and way share an id It turns out the _type parameter to the Elasticsearch [multiget](https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-multi-get.html) API does not allow an array of possible values. We were depending on its ability to search multiple types to allow searching for OSM nodes and ways. But, since this doesn't work we essentially have to do it ourselves. There is also the problem that OSM nodes and ways share an ID space. So a gid such as `osm:venue:5` could in theory correspond to 2 records. It seems like the only nice thing to do in that case is return both results. This PR "unrolls" such queries. For example, in the case of `osm:venue:5`, the sanitisers will return the following array of objects to be turned into multiget queries: ``` [{ id: 5, types: ["osmway", "osmnode"] }] ``` Before, this would turn into a multiget query with only one entry, like this: ``` { "docs": [ { "_index": "pelias", "_type": [ " osmnode", "osmway" ], "_id": 5 } ] } ``` now it would look like this: { "docs": [ { "_index": "pelias", "_type": "osmnode", "_id": 5 }, { "_index": "pelias", "_type": "osmnode", "_id": 5 } ] } TLDR you might get back more records from /place than the number of ids you specified, but at least you will definitely get back what you are looking for.	9 years ago
Peter Johnson	687c26555f	installing pelias/scripts no longer required	9 years ago
Peter Johnson a.k.a. insertcoffee	f06fe8880f	Merge pull request #335 from pelias/remove_tiebreakers remove all references to tiebreaking groovy scripts	9 years ago
Peter Johnson a.k.a. insertcoffee	2995c82b82	Merge pull request #340 from pelias/improved_labels more label improvements	9 years ago
Peter Johnson a.k.a. insertcoffee	928f06121f	Merge pull request #338 from pelias/improved_population_popularity_boosting better population and popularity boosting	9 years ago
Peter Johnson	85a51bc48a	more usa test cases	9 years ago
Peter Johnson	d93f9c1803	more new zealand test cases	9 years ago
Peter Johnson	7cf8638795	more australian and thai test cases	9 years ago
Peter Johnson	5cc432ae50	more indian test cases	9 years ago
Peter Johnson	b5b8647f0c	more label improvements	9 years ago
Peter Johnson	8ceac4cbb0	change modifier function from sqrt() to log1p() and adjust max_boost	9 years ago
Peter Johnson	2ce249ad30	adjust balance between population/popularity and focus	9 years ago
Peter Johnson	481a65a4f9	population twice as important as popularity	9 years ago
Peter Johnson	cb679888ff	better population and popularity boosting	9 years ago
Peter Johnson a.k.a. insertcoffee	bfd2720681	Merge pull request #336 from pelias/remove_docs remove generated ciao docs	9 years ago
Peter Johnson	ea7a177934	remove generated ciao docs	9 years ago
Peter Johnson	56591abe97	remove all references to tiebreaking groovy scripts	9 years ago
Julian Simioni	05a840753d	Add comment for confidence score function	9 years ago
Peter Johnson a.k.a. insertcoffee	a03a18ef67	Merge pull request #333 from pelias/http_status_codes http friendly status codes	9 years ago
Julian Simioni	cc9cdfca22	Add link to GitHub documentation on /v1/ "landing page" Fixes #234	9 years ago
Peter Johnson	c609ae3ccf	http friendly status codes. resolves #321	9 years ago
Peter Johnson a.k.a. insertcoffee	38b53e7264	Merge pull request #331 from pelias/fix_ciao_tests fix ciao tests	9 years ago
Peter Johnson	74c9f2e42d	merge master	9 years ago
Peter Johnson a.k.a. insertcoffee	4241cdbc85	Merge pull request #330 from pelias/clean-up-ciao-tests Fix low hanging fruit in ciao tests	9 years ago
Peter Johnson	51858781a9	fix ciao tests	9 years ago
Julian Simioni	ab006054f8	Fix low hanging fruit in ciao tests Quite a few of our tests simply had old error messages as expectations, or expected output in an array that was correct but in the wrong order. Those are all fixed.	9 years ago
Julian Simioni	aa896dab93	Merge branch 'unit-tests-for-type-mapping-helper'	9 years ago
Stephen Hess	4be987045e	updated tests to reflect master type_mapping	9 years ago
Stephen Hess	44f335bbe0	Merge branch 'master' into unit-tests-for-type-mapping-helper	9 years ago
Stephen Hess	0a142a99d9	Merge branch 'master' into unit-tests-for-type-mapping-helper	9 years ago
Stephen Hess	a26bf0b9a8	switched to Math.max to avoid the explicit conditional	9 years ago
Stephen Hess	dcadc7832e	add focus:scale to /search requests with just viewport.*, calculated from bounding box diagonal, minimum of 1	9 years ago
Peter Johnson a.k.a. insertcoffee	3629a49dc1	Merge pull request #322 from pelias/remove_cache_control_headers remove cache control headers	9 years ago
Peter Johnson	a209024143	remove cache control headers	9 years ago
Peter Johnson a.k.a. insertcoffee	a77b7045fe	Merge pull request #319 from pelias/remove_microtime remove microtime module	9 years ago
Peter Johnson	b5b3c41ba1	remove microtime module	9 years ago
Julian Simioni	fef92bd34b	Remove the geoname _type from all layers except venue We can't distinguish between geonames of different layers due to an ambiguity in our Elasticsearch schema that we unfortunately won't be able to fix for a few weeks. So, while it's technically true that there are countries, cities, etc contained in the geonames dataset, it's still better for now to remove geonames from these layers. We have good coverage of most coarse layers from quattroshapes alone, so the impact isn't too bad.	9 years ago
Peter Johnson a.k.a. insertcoffee	ef8206be6c	Merge pull request #314 from pelias/remove-redundant-ids-array-check removed redundant is-ids-an-array check	9 years ago
Julian Simioni	57fb960471	Set centerpoint of viewport in search query This simply reuses the focus:point:{lat\|lon} variables, but sets them using the centerpoint of the viewport. Eventually we should calculate a radius and use that here.	9 years ago
Julian Simioni	06b44e5355	Disallow specifying both focus.point and focus.viewport	9 years ago
Julian Simioni	9dcc296a70	Add focus.viewport rect sanitizer to search	9 years ago
Julian Simioni	36925ca874	Remove bbox sanitizer functionality tests from search sanitizer tests Full functionality testing for the bbox sanitizer should be in the sanitizer/_geo_common tests. In the search sanitizer tests, only testing the inclusion of the correct sanitizers, and the interactions between them, really seems to make sense.	9 years ago
Julian Simioni	29a78db403	Add null island bounding box test	9 years ago
Julian Simioni	db8f98a51f	Remove completed @todo	9 years ago

1 2 3 4 5 ...

829 Commits (b188a2046c876e124d8f6fda4fc7ce16b1a04a8c) All Branches Search

829 Commits (b188a2046c876e124d8f6fda4fc7ce16b1a04a8c)

All Branches