Vagrantfile - remove Ubuntu 18

Merge pull request #3067 from lonvia/python-search-api

Python implementation of Search API

make code backwards-compatible with older Python versions

remove tests for old PHP cli commands

fix new linting warning

fix various failing BDD tests

add support for search by houenumber

ignore name rankings without variants

properly close connections when shutting down starlette

fix geocodejson address assignment

The categories should be assigned the address part with the
highest address rank not lowest.

add server fronting for search endpoint

This also implements some of the quirks of free-text search of the
V1 API, in particular, search for categories and coordinates.

add timestamps to HTML debug output

mingle names from linked places into results

switch CLI search command to python implementation

make localisation of results explicit

Localisation was previously done as part of the formatting but might
also be useful on its own when working with the results directly.

add API functions for search functions

Search is now split into three functions: for free-text search,
for structured search and for search by category. Note that the
free-text search does not have as many hidden features like
coordinate search. Use the search parameters for that.

implement actual database searches

implement search builder

implement token assignment

add query analyser for legacy tokenizer

add factory for query analyzer

query analyzer for ICU tokenizer

add ci-test for taginfo file generation

add data structure for tokenized query

docs: add installation instruction for dkjson package

Add script to generate taginfo project description

Merge pull request #3064 from lonvia/clicmd-debug-output

Enable debug output on command line

enable debug output on command line

Merge pull request #3063 from lonvia/variable-parameters

Rework how search parameters are handed to the Python API

fix use of subquery in reverse

move zoom_to_rank computation to extra file

add tests for parameter converter

Merge pull request #3062 from lonvia/enable-psycopg

Add support for psycopg 3 in the frontend

switch API parameters to keyword arguments

This switches the input parameters for API calls to a generic
keyword argument catch-all which is then loaded into a dataclass
where the parameters are checked and forwarded to internal
function.

The dataclass gives more flexibility with the parameters and makes
it easier to reuse common parameters for the different API calls.

actions: run tests against psycopg

enable API use with psycopg 3

Merge pull request #3050 from mtmail/tiger-check-if-database-frozen

when adding Tiger data, check first if database is in frozen state

when adding Tiger data, check first if database is in frozen state

Merge pull request #3054 from lonvia/add-amenity-to-documentation

Docs: reintroduce amenity parameter for structured search

docs: reintroduce amenity parameter for structured search

Merge pull request #3006 from biswajit-k/generalize-filter

generalize filter function for sanitizers

Merge remote-tracking branch 'upstream/master'

Merge pull request #3030 from lonvia/interpolation-corner-cases

Fix a number of corner cases with interpolation splitting

fix a number of corner cases with interpolation splitting

Snapping a line to a point before splitting was meant to ensure
that the split point is really on the line. However, ST_Snap() does
not always behave well for this case. It may shorten the interpolation
line in some cases with the result that two points housenumbers
suddenly fall on the same point. It might also shorten the line down
to a single point which then makes ST_Split() crash.

Switch to a combination of ST_LineLocatePoint and ST_LineSubString
instead, which guarantees to keep the original geometry. Explicitly
handle the corner cases, where the split point falls on the beginning
or end of the line.

use place_to_be_deleted when force deleting objects

Merge pull request #3023 from lonvia/lookup-api

Python implementation of Lookup API

switch CLI lookup command to Python implementation

add lookup call to server glue

add tests for new lookup API

rename lookup() API to details and add lookup call

The initial plan to serve /details and /lookup endpoints from
the same API call turned out to be impractical, so the API now
also has deparate functions for both.

python lookup: add function for simple lookups

python lookup: factor out finding in tables into own function

Merge remote-tracking branch 'upstream/master'

Merge pull request #3021 from lonvia/readd-postalcode-relations

Flex style: reinstate postalcode relations

bdd: disable detail tests searching by place ID

Place IDs are not stable and cannot be used in tests.

flex style: reinstate postcode boundaries

Postcode boundaries don't have a name, so need to be imported
unconditionally.

generalize filter for sanitizers

Merge pull request #3019 from lonvia/add-data-postprocessing

Add postprocessing to add-data function

call osm2pgsql postprocessing flush_deleted_places() when adding data

Merge pull request #3020 from lonvia/reverse-api

Python implementation of reverse

limit results for country lookup

add BDD tests for new layers parameter

python reverse: add support for point geometries in interpolations

add wsgi entry point for falcon server

add wsgi entry point for starlette

uvicorn needs a parameter-free function to start.

ignore broken data in interpolation table

disable prepared statements

Prepared statements do not work well with the partial indexes that
Nominatim uses because all Python constants are replaced with
parameters. A query like:

  placex.select().where(placex.c.rank_address.between(4, 25)

gets translated into a prepared query with two parameters:

  SELECT * FROM placex WHERE rank_address BETWEEN %s and %s

And this does not work with a partial index of:

  CREATE INDEX on placex(geometry) WHERE rank_address between 4 and 25

switch reverse CLI command to Python implementation

make sure PHP and Python reverse code does the same

The only allowable difference is precision of coordinates. Python uses
a precision of 7 digits where possible, which corresponds to the
precision of OSM data.

Also fixes some smaller bugs found by the BDD tests.

add server glue for reverse API call

add output formatters for ReverseResults

These formatters are written in a way that they can be reused for
search results later.

ignore NotImplementedErrors when compiling SQL

factor out layer checks in reverse function

python: implement reverse lookup function

The implementation follows for most part the PHP code but introduces an
additional layer parameter with which the kind of places to be returned
can be restricted. This replaces the hard-coded exclusion lists.

add python implementation of reverse

This adds an additional layer parameter and slightly changes the
queries to do more efficient lookups for large area features.

api: make details parameter optional

api: generalize error handling

Return a consistent error response which takes into account the chosen
content type. Also adds tests for V1 server glue.

split SearchResult type

Use adapted types for the different result types. This makes it
easier to have adapted output formatting and means there are only
result fields that are filled.

Merge pull request #3013 from mtmail/changelog-4-2-x

update ChangeLog to verison 4.2.2

update ChangeLog to verison 4.2.2

Merge pull request #3011 from lonvia/fix-flex-scripts

Fix extra tag handling in some flex scripts

Merge remote-tracking branch 'upstream/master'

Merge pull request #3010 from lonvia/adapt-zoom-for-reverse

Minor adjustments to reverse zoom translation

remove more tags from full style

The full style should only save the necessary tags needed for
processing.

fix handling of unused extra tags

The tags can only be moved to extra tags after the main tags have been
handled.

minor adjustment to reverse zoom translation

Add a 'village' zoom level at 13 between town and neighbourhood
and a all locality-like objects for zoom 15. These zoom levels had
the same behaviour as the lower level so far. However, the distinction
for village and locality may be useful at times.

Merge remote-tracking branch 'upstream/master'

Merge pull request #3003 from lonvia/rework-bdd-api-tests

Reorganize code around BDD API tests and extend reverse API tests

bdd: be more fuzzy with coordinate comparisons

avoid recent Python dialect

fix polygon simplification in reverse results

polygon_threshold has never really worked for reverse.

bdd: extend reverse API tests for format checks

Reorganise the API reverse tests and extend the checks for the
output format, testing for all expected fields.

bdd: add tests for valid debug output

bdd: replace old formatting strings

bdd: more format checks for reverse XML

bdd: new step variant 'result contains in field'

This replaces the + notation for recursing into result dictionaries.

bdd: factor out computation of result to-check lists

bdd: replace property_list construct with standard check functions

bdd: remove special case for osm_type field

The fuzzy field check hide cover formatting errors. Use 'osm' when
only caring about the conent.

bdd: reorganise field comparisons

Move comparision on Field values from assert_field() into a
comparator class. Replace BadRowValueAssert with a simpler
check_row() function.