pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Stephen Hess	c19d56697e	add _private sanitizer	9 years ago
Stephen Hess	55e31af238	extract truthy sanitizer to own file	9 years ago
Stephen Hess	2f5a5b1972	refactored _details tests to only _details sanitizer	9 years ago
Stephen Hess	e52c9e4ab9	moved details tests out to separate file	9 years ago
Julian Simioni	96c58e8658	Rename /reverse lat/lon to point.lat/lon	9 years ago
Julian Simioni	3176a61596	Rename /search lat/lon to focus.point.lat/lon	9 years ago
Julian Simioni	6c1cfbb4ad	Compare latitude values only, instead of full sanitiser output	9 years ago
Julian Simioni	5d69c8016b	Remove lat/lon params from tests that don't use it This just removes clutter and possible confusion from our test cases.	9 years ago
Julian Simioni	fd3b12cfa9	Remove all references to non-existent zoom parameter	9 years ago
Julian Simioni	040871cbe4	Whitespace	9 years ago
Peter Johnson	c0f20aabd3	refactor: clean up sorting conditions	9 years ago
Peter Johnson	36185197aa	move query defaults to API repo	9 years ago
Peter Johnson	d568c031a3	move stringify logic to tests, code clean up	9 years ago
Peter Johnson	fa209c9b50	add autocomplete route, further query clean up	9 years ago
Peter Johnson	2f8169ac10	remove unused sort conditions	9 years ago
Peter Johnson	22322eaa32	change param:input to param:text	9 years ago
Peter Johnson	1f13bafab0	refactor reverse & tests	9 years ago
Julian Simioni	139cb22127	Remove layers limiting on queries with few tokens Primarily as a performance optimization, but also to attempt to return more relevant results, only admin and POI layers were queried when the text input consisted of only one or two tokens, and there weren't any numbers. However as shown in #194 that is a bit too optimistic, mostly in contries other than the USA. Fixes #194	9 years ago
Julian Simioni	cb2f1172e3	Use text parameter in ciao tests	9 years ago
Harish Krishna	86424b8bbe	Rename `/search?input`, replacing it with '/search?text' parameter instead changing parameter name input to text https://github.com/pelias/api/issues/172 Rebased by @orangejulius	9 years ago
Julian Simioni	d9ede52e1b	Remove extra input paremter from reverse unit tests Not strictly required for this change, but I noticed there was an input parameter to lots of unit tests for the reverse endpoint. Reverse doesn't take an input(or text) parameter at all, so these are just extra, and probably came from a copy/paste in the tests.	9 years ago
Julian Simioni	dc6ed3d6a9	Remove unused suggest endpoint unit tests	9 years ago
Julian Simioni	103a52cff5	Calculate intersection of types requested by source and layers params	9 years ago
Julian Simioni	b4e7793426	Fix reference to geoname This is just in a unit test, so it technically passes, but geonames is not a valid layer option (geoname, singular, is used instead), so it's best to correct.	9 years ago
Julian Simioni	3383fa0321	Add source parameter sanitiser	9 years ago
Julian Simioni	2ebc935295	Rename query/indidces to query/types It didn't really contain a list of indices.	9 years ago
Julian Simioni	6f92189519	Remove default_layers_set flag! After refactoring, this flag is no longer needed, as all areas of the code that care about layers do so by setting a key within clean.types, and then the types helper intelligently combines those together later.	9 years ago
Julian Simioni	258fa4390e	Move cmd.type setting logic to types helper	9 years ago
Julian Simioni	7a6ac8541b	Separate concerns of address parser The address parser currently does two things: 1.) make some intelligent guesses as to possible admin regions to explicitly search against to improve the quality of results returned 2.) make some intelligent guesses as to when no part of the query needs to search against anything other than admin regions. This somewhat improves the quality of results returned but mostly improves the speed of the Elasticsearch query since it's searching significantly fewer recoords. These two concerns are now split into two separate methods within the query_parser helper module. They are mostly independent today, but don't have to be in the future.	9 years ago
Julian Simioni	bc8eabe4e6	Remove loops and many global checks from sanitiser tests Modifying these sanitiser tests became extremely hard because almost all of them were looping over lots of individual test cases, which places assumptions about the common behavior of potentialy very different test cases, as well as making assertions about huge swaths of output when only a small amount of that output was really under test. Hopefully these changes will make our tests easier to modify, and not really lose any ability to catch bugs.	9 years ago
Julian Simioni	1fae622c39	Remove layer configuration based on address parsing This code doesn't seem like it will be triggered very often (due to it comapring space delimited words with comma delimited words from the text field), and also has the potential to cause quite a bit of weird behavior.	9 years ago
Julian Simioni	82d3a6603c	Store sanitised types from layers parameter This moves the list of types created by sanitising the layer API parameter from clean.layers to clean.types.from_layers. In subsequent commits, types created from address parsing, and the yet-to-be-implemented source parameter will also live in the clean.types object. This will allow moving logic to set cmd.type out of controllers, and into separate logic that can be a littler smarter. Also, it will no longer require the clean.default_layers_set flag to be passed all around like a nasty global variable.	9 years ago
Julian Simioni	e71fb7cfd4	Whitespace	9 years ago
Peter Johnson	f5e182c63b	fix tests, clean by moving fixtures to fixtures dir	9 years ago
Diana Shkolnikov	2eccfc5c9a	Fix unit test	9 years ago
Diana Shkolnikov	f7044d2435	Add content for `attribution` static endpoint Fixes #212	9 years ago
Julian Simioni	a479de7527	Calculate intersection of types requested by source and layers params	9 years ago
Julian Simioni	eed010835b	Fix reference to geoname This is just in a unit test, so it technically passes, but geonames is not a valid layer option (geoname, singular, is used instead), so it's best to correct.	9 years ago
Julian Simioni	3479325fa8	Add source parameter sanitiser	9 years ago
Julian Simioni	24349a3839	Rename query/indidces to query/types It didn't really contain a list of indices.	9 years ago
Julian Simioni	3ff1752434	Remove default_layers_set flag! After refactoring, this flag is no longer needed, as all areas of the code that care about layers do so by setting a key within clean.types, and then the types helper intelligently combines those together later.	9 years ago
Julian Simioni	1573f3a3fa	Move cmd.type setting logic to types helper	9 years ago
Julian Simioni	b9152dbe29	Separate concerns of address parser The address parser currently does two things: 1.) make some intelligent guesses as to possible admin regions to explicitly search against to improve the quality of results returned 2.) make some intelligent guesses as to when no part of the query needs to search against anything other than admin regions. This somewhat improves the quality of results returned but mostly improves the speed of the Elasticsearch query since it's searching significantly fewer recoords. These two concerns are now split into two separate methods within the query_parser helper module. They are mostly independent today, but don't have to be in the future.	9 years ago
Julian Simioni	d35544cdf4	Remove loops and many global checks from sanitiser tests Modifying these sanitiser tests became extremely hard because almost all of them were looping over lots of individual test cases, which places assumptions about the common behavior of potentialy very different test cases, as well as making assertions about huge swaths of output when only a small amount of that output was really under test. Hopefully these changes will make our tests easier to modify, and not really lose any ability to catch bugs.	9 years ago
Julian Simioni	ad8db9b8e9	Remove layer configuration based on address parsing This code doesn't seem like it will be triggered very often (due to it comapring space delimited words with comma delimited words from the text field), and also has the potential to cause quite a bit of weird behavior.	9 years ago
Julian Simioni	4037c49c4b	Store sanitised types from layers parameter This moves the list of types created by sanitising the layer API parameter from clean.layers to clean.types.from_layers. In subsequent commits, types created from address parsing, and the yet-to-be-implemented source parameter will also live in the clean.types object. This will allow moving logic to set cmd.type out of controllers, and into separate logic that can be a littler smarter. Also, it will no longer require the clean.default_layers_set flag to be passed all around like a nasty global variable.	9 years ago
Julian Simioni	fb0cf514e9	Whitespace	9 years ago
Diana Shkolnikov	562338635c	Confidence score calculation Fixes pelias/pelias#120	9 years ago
Julian Simioni	63d0447b25	Update ciao tests for /v1	9 years ago
Diana Shkolnikov	1d8db6e5d1	Change type back to layer... oooops	9 years ago

... 6 7 8 9 10 ...

672 Commits (county-fallback)