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
ff4562edb6
Merge pull request #380 from pelias/fix_confidence_score_testing
...
Check that hit.address exists
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
Julian Simioni
d0e2c14738
Check that hit.address exists
...
Fixes #379
9 years ago
Diana Shkolnikov
ac43d6c226
Merge pull request #376 from pelias/dedupe-post-es
...
Add dedupe middleware
9 years ago
Diana Shkolnikov
af7abce68f
Merge remote-tracking branch 'origin' into dedupe-post-es
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
36035131f0
Merge pull request #368 from pelias/postal-code-strip-zeros
...
Update addressit module to not strip 0s from zipcodes
9 years ago
Julian Simioni
7e569463f0
Merge pull request #369 from pelias/fix-postalcode-confidence-score
...
Fix postalcode confidence score
9 years ago
Diana Shkolnikov
ed2d8bea72
Update package.json
9 years ago
Grant Heffernan
f3915f58a9
deploy to prod_build when staging is updated, not production
9 years ago
Grant Heffernan
c9fce33c44
circle.yml: do not need acceptance tests here
9 years ago
Grant Heffernan
8cf49b0ac5
automatically deploy api to prod_build when merged to production
9 years ago
Julian Simioni
c6c547f6a2
Log expected and actual zipcode when computing confidence
9 years ago
Julian Simioni
bb3fede3c0
Fix references to address zip
9 years ago
Diana Shkolnikov
804a520839
Fix github link
9 years ago
Diana Shkolnikov
500835a4c1
Update addressit module to not strip 0s from zipcodes
9 years ago
Grant Heffernan
f722d3fcd1
circleci: remove production branch automated deployment for the time being
9 years ago
Peter Johnson a.k.a. insertcoffee
0ce471a166
Merge pull request #351 from pelias/local_naming_conventions
...
german street/number naming convention
9 years ago
Peter Johnson a.k.a. insertcoffee
a0e174fad8
Merge pull request #356 from pelias/improved_address_schema
...
update address matching analyzers.
9 years ago
Peter Johnson
c17a0dbc54
increase postcode weights, add postcode to leftovers
9 years ago
Peter Johnson
ac01d72477
update address matching analyzers. related https://github.com/pelias/schema/pull/77
9 years ago
Grant Heffernan
fdcdea3258
simplify circle deployment
9 years ago
Peter Johnson a.k.a. insertcoffee
b5607d5371
Merge pull request #352 from pelias/per_query_config
...
one config per query
9 years ago
Grant Heffernan
90d1afbc09
- separate prod and prod_build env deploys
...
- run acceptance tests after deploy
9 years ago
Grant Heffernan
71636cc8ce
Merge pull request #353 from pelias/circleci
...
Circleci
9 years ago
Grant Heffernan
a9fe638dfb
acceptance tests run against dev
9 years ago
Grant Heffernan
a8245efc33
deploy master to dev
9 years ago
Grant Heffernan
1886b98510
Merge pull request #326 from pelias/circleci
...
circle.yml: automate api deployments for prod/prod_build
9 years ago
Grant Heffernan
8250989025
Revert "should only need search api key"
...
This reverts commit 84bae8226b
.
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
6fdc581a6c
Merge pull request #347 from pelias/finish-rename-to-text-parser
...
Finish Renaming address_parser to text_parser
9 years ago
Julian Simioni
11a5b638ae
Merge pull request #345 from pelias/use-tap-dot
...
Use tap-dot
9 years ago
Grant Heffernan
84bae8226b
should only need search api key
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
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
Grant Heffernan
f5d823ddc2
fix circle.yml typo
9 years ago
Julian Simioni
098c3c74ab
Use tap-dot when running tests
9 years ago
Julian Simioni
b1f615479c
Replace tap-spec module with tap-dot
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