pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Stephen Hess	f912cb322c	added USA county test	9 years ago
Stephen Hess	24201188c5	broke country-specific label generation tests out to separate files for easier maintenance	9 years ago
Julian Simioni	0e78167f17	[/place] Accept any valid layer for geonames Instead of checking if the source/layer combination for geonames is valid, always use the geoname type if the source is set to geonames, regardless of which layer is in the gid. This helps us better handle the case where people directly take gids from our API responses. Note: this functionality shouldn't be permanent and should be removed once our new source/layer mapping system is in.	9 years ago
Julian Simioni	18de8b5699	Remove unused test examples	9 years ago
Julian Simioni	435104842d	Fix failing Ciao tests They were expecting certain parts of the response to not be there when an error is returned, but that's overspecifying the result in my opinion.	9 years ago
missinglink	6711aa45e0	simplify pass 1	9 years ago
missinglink	3b4d566b11	add unit test	9 years ago
Diana Shkolnikov	df93be7543	Fix deduping for non-ascii strings	9 years ago
Peter Johnson	db5a78623a	disable viewport scale	9 years ago
Peter Johnson	f1ead56e60	disable viewport scale	9 years ago
Julian Simioni	87f6f32985	Use tape's throws where appropriate	9 years ago
Julian Simioni	fcc9237993	Use tape's doesNotThrow where appropriate	9 years ago
Julian Simioni	b5be8dea42	Remove duplicate test	9 years ago
Julian Simioni	5a84afa539	Add test of parsing hit with no address object In #369 I said I wouldn't bother testing for the case of the address object on hit being absent, but it turns out that does sometimes happen.	9 years ago
Diana Shkolnikov	9fa5fc5a77	calcSize became middleware (exposed and fixed bug in query defaults)	9 years ago
Diana Shkolnikov	42d940f8c8	Add simple normalizer (lowercase + remove punctuation)	9 years ago
Diana Shkolnikov	54187dde67	Add dedupe middleware Dedupe middleware removes __exact__ dupes and truncates the results to the specified size.	9 years ago
Diana Shkolnikov	500835a4c1	Update addressit module to not strip 0s from zipcodes	9 years ago
Peter Johnson	49d37eafed	one config per query	9 years ago
Peter Johnson	4e43310288	german street/number naming convention	9 years ago
Julian Simioni	db895c05b7	Rename `from_address_parser` key to `from_text_parser`	9 years ago
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
Stephen Hess	d732021d5e	fixed tests	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	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
Peter Johnson	c609ae3ccf	http friendly status codes. resolves #321	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
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
Peter Johnson	a209024143	remove cache control headers	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

1 2 3 4 5 ...

422 Commits (modify-label-generation-for-wof)