pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
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	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	56591abe97	remove all references to tiebreaking groovy scripts	9 years ago
Stephen Hess	4be987045e	updated tests to reflect master type_mapping	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
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
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
Julian Simioni	b84652ba20	Refactor sanitize_coord parameter handling This should help reduce duplication when passing values in, as the key won't have to be specified twice. Also, the parameters are in the same order as the other sanitize_* methods.	9 years ago
Stephen Hess	00d033101b	removed redundant is-ids-an-array check (already covered by _single_scalar_parameter sanitizer)	9 years ago
Julian Simioni	415029e02b	Use groups sanitiser in sanitize_rect and sanitize_point	9 years ago
Julian Simioni	aa08fb9772	Add optional and required group parameter sanitizers	9 years ago
Julian Simioni	f3ba5411cd	Sort test file names	9 years ago
Stephen Hess	02792de2dc	added unit tests for verifying valid mapping between sources and layers	9 years ago
Peter Johnson	0918d3031f	remove duplicate test	9 years ago
Stephen Hess	cb7d22df91	switched ES lookup field from "type" -> "_type", fixes issue with /place lookup on Eiffel Tower gid	9 years ago
Peter Johnson	84dfa0f45e	add custom singapore schema, it is unusual as it is both a city and a country	9 years ago
Peter Johnson	87ce3bd92e	flip lon values for boundary.rect	9 years ago
Peter Johnson	3a54a8b32f	fix label issues	9 years ago
Peter Johnson	2b9c3d79aa	rename outputGenerator -> labelGenerator	9 years ago
Diana Shkolnikov	ce32802368	Hotfix for confidence exception	9 years ago
Diana Shkolnikov	62315a1775	Revert defaults.json change	9 years ago
Julian Simioni	2811569bcd	Switch to source:layer:id ids format in /place	9 years ago
Julian Simioni	d4fff26574	Derive type, source, and layer list instead of hardcoding	9 years ago
Julian Simioni	d11d185429	Fix missing space in error message	9 years ago
Julian Simioni	2505e92a62	Expect an array of types from _ids sanitiser This doesn't have any effect by itself but allows for the 3-part gid sanitiser to possibly return multiple types (i.e. in the case of osm:venue:1000)	9 years ago
Julian Simioni	aabca1569c	Fix old reference to /doc endpoint	9 years ago
Julian Simioni	fa44effdac	Add geoname type to most layers The Geonames dataset includes lots of different kinds of places, so add them to the mapping.	9 years ago
Julian Simioni	8b1037d7c8	Separate real layers and alias layers into separate objects	9 years ago
Julian Simioni	68c9661c70	Remove all mappings other than helper/type_mapping	9 years ago
Julian Simioni	4df0f98b14	Add type list, and raw mappings to/from source/layer and type	9 years ago
Julian Simioni	dc623d5af7	Send 3 part gid (source, layer, id) in GeoJSON responses	9 years ago
Stephen Hess	56d8600218	modified warning message for brevity	9 years ago
Stephen Hess	0f6d08c0ec	set boundary.circle.radius to default if caller did not supply it	9 years ago
Stephen Hess	947797f41e	added warning if any of boundary.circle.lat/lon/radius are supplied + tests	9 years ago
Julian Simioni	ebb3f33b2d	Remove unused test file	9 years ago
Peter Johnson	80f419846d	admin boost for sqrt(popularity), autocomplete focus not restricted to phrase matches	9 years ago
Diana Shkolnikov	2b22d611bd	Fix lost confidence score	9 years ago

... 4 5 6 7 8 ...

661 Commits (1216ca783d3a2077993a3739c50a3b5ea0274cac)