split db_searches moving each class in its own file

Merge pull request #3765 from lonvia/update-ui-docs

Update instructions for UI integration

update instructions for UI integration

Switches from defaulting to forwarding to UI to only forwarding
when requested. This avoids issues with auto-forwarding illegal URLs.
Also adapts to the much simplified nginx configuration.

Merge pull request #3764 from lonvia/update-importance

'refresh --importance' also needs to refresh importances in search_name table

remove duplicated test

There is a more extensive test of recompute_importance with
result check in test_refresh_wiki_data.py

Merge pull request #3762 from lonvia/remove-gazetteer-output-support

Remove support for deprecated gazetteer osm2pgsql output

rewrite importances in search_name after updating in placex

remove support for deprecated gazetteer osm2pgsql output

Merge pull request #3747 from anqixxx/fix-special-phrases-filtering

Special Phrases Filtering: Add Command Line Functionality

Added skip for when min =0

Added default min = 0 argument for private functions

empty

Refactored min and associated tests to follow greater than or equal to logic, so that min=0 accounted for no filtering

r

Added command line (default 0) min argument for minimum filtering, updated args.py to reflect this

Merge pull request #3748 from lonvia/airports

Improve finding airports by their codes

improve result cutting when a POI comes out with top importance

only rerank results if there is more than one

With one result order is obvious.

reduce influence of query rematching a little bit

when rematching only distinguish between perfect, somewhat and bad match

add FAA codes (US version of IATA codes) for airports

work around bogus type error in latest starlette

fix missing optional return

Merge pull request #3744 from lonvia/add-unnamed-cemetries

Include unnamed cemetaries in POIs

add landuse=cemetery as POI even when unnamed

Merge pull request #3742 from lonvia/korean-defaults

Remove English as default language for South Korea

adapt to changed loop handling of pytest_asyncio

remove English as default language for South Korea

Merge pull request #3741 from dave-meyer/patch-1

docs: Added missing code span for search API parameter value

docs: Added missing code span for search API parameter value

Merge pull request #3710 from anqixxx/fix-special-phrases-filtering

Fix special phrases filtering

Added mypy ignore fix for logging.py (library change), as well as quick mac fix on mem.cached

Merge pull request #3709 from anqixxx/update-readme

Improve README formatting and add install steps

Added testing to test get classtype pairs in import special phrases

Made the limit configurable with an optional argument, updating the testing as well to reflect this. default is now 0, meaning that it will return everything that occurs more than once. Removed mock database test, and got rid of fetch all. Rebased all tests to monkeypatch

Removed class type pair getter that used style sheets from both spi_importer and the associated testing function

Removed magic mocking, using monkeypatch instead, and using a placex table to simulate a 'real database'

Filter special phrases by style and frequency to fix #235

Improve README formatting and add install steps, adding a general cloning step before the virtual environment. This would have been helpful for me during Nominatim setup

tweak full count cut-off (as per deployment on osm.org)

Merge pull request #3732 from lonvia/exclude-country-from-direction-penalty

Exclude address searches with country from direction penalty

exclude address searches with country from direction penalty

Countries are not adequately represented by partial term counts.

Merge pull request #3722 from emmanuel-ferdman/master

resolve datetime deprecation warnings

Merge pull request #3726 from lonvia/revert-json-format-change

Revert accidental change in json output format

revert accidental change in json output format

resolve datetime deprecation warnings

Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com>

Merge pull request #3719 from lonvia/query-direction

Estimate query direction

add a comment about the precomputed denominator

add direction penalties

Direction penalties are estimated by getting the name to address
ratio usage for each partial term in the query and computing the
linear regression of that ratio over the entire phrase. Or to put
it in ither words: we try to determine if the terms at the beginning
or the end of the query are more likely to constitute a name.

Direction penalties are currently used only in classic name queries.

reorganise token reranking

As the reranking is about changing penalties in presence of other
tokens, change the datastructure to have the other tokens readily
avilable.

ensure that bailout-check is done after each iteration

replace use of range when computing word list

use iterator instead of list to go over partials

move partial token into a separate field in the query struct

There is exactly one token to be expected and the token is usually
present.

Merge pull request #3716 from lonvia/github-cache-osm2pgsql-binary

Github actions: cache compiled osm2pgsql binary

For the tests on Ubunutu 22-04 we need to compile osm2pgsql because the version they ship is too old. This adds caching of the compiled binary, so that we don't need to recompile for each CI run. Together with the new BDD tests that shaves around 10 min off a CI run.

actions: cache compiled osm2pgsql binary

Merge pull request #3714 from lonvia/postcode-update-without-project-dir

Change postcode update function to work without a project directory

Merge pull request #3715 from lonvia/demote-tags-to-fallbacks

Demote historic and tourism=attraction to fallback tags

demote historic and tourism=attraction to fallback tags

allow updating postcodes without a project directory

Postcodes will then be updated without looking for external postcodes.

Merge pull request #3713 from lonvia/bdd-pytest-db-test

Move BDD tests to pytest-bdd

update documentation for BDD tests

remove usage of behave

add BDD tests for DB

improve table comparison

move database setup to generic conftest.py

remove old behave osm2pgsql BDD tests

implement BDD osm2pgsql tests with pytest-bdd

enable python-bdd for github actions

restrict coordinate output to 7 digits

replace behave BDD API tests with pytest-bdd tests

Merge pull request #3687 from asharmalik19/test-linked-places-language

test: linked places expand default language names

Update linked places name test

Clean up test scenario by removing extra language variations and
improving table readability.

test: linked places expand default language names

Add failing test for issue #2714 to verify default language expansion

Merge pull request #3702 from lonvia/remove-tokenizer-dir

Remove automatic setup of tokenizer directory

So far the tokenizer factory would create a directory for private data for the tokenizer and then hand in the directory location to the tokenizer.

ICU tokenizer doesn't need any extra data anymore, so it doesn't make sense to create a directory which then remains empty. If a tokenizer needs such a directory in the future, it needs to create it on its own and make sure to handle the situation correctly where no project directory is used at all.

remove automatic setup of tokenizer directory

ICU tokenizer doesn't need any extra data anymore, so it doesn't
make sense to create a directory which then remains empty. If a
tokenizer needs such a directory in the future, it needs to create
it on its own and make sure to handle the situation correctly where
no project directory is used at all.

Merge pull request #3700 from lonvia/ignore-inherited-addresses

Ignore POIs with inherited addresses for the address layer

ignore POIs with inherited addresses for the address layer

We know that there is a building which describes the address as a
polygon and is therefore more suitable.

Merge pull request #3686 from astridx/output_names

Output names as setting

Merge pull request #3675 from TuringVerified/generic-preprocessors

Add generic preprocessors

Remove unnecessary assert statement, Fix regex_replace docstring and simplify regex_replace

Update documentation, optimise regex_replace, add tests

Add generic preprocessor

add instructions for pip package upload

prepare release 5.1.0

Merge pull request #3695 from TuringVerified/doc-dependencies

[Small fix] Add documentation to install extras for mkdocstrings

Add mkdocstrings extra

Merge pull request #3693 from lonvia/remove-unused-sql

Remove SQL function for address lookup

output names as setting

Merge pull request #3692 from lonvia/word-lookup-variants

Avoid matching penalty for abbreviated search terms

remove SQL function for address lookup

This is now done in Python.

Merge pull request #3691 from lonvia/more-search-tweaks

More tweaks to search wights

adapt tests to extended results

add lookup word to variants in word table

do not write any word counts on initial word insert

do not save word counts of 1

This is the default setting, which will be assumed when the count is
missing.

always use lookup when requested

Doesn't seem to cause any issues in production.

increase allowable address counts

more agressively reduce expected count for multi-word terms

Improves searching of non-latin scripts with forced token spaces.

Merge pull request #3690 from lonvia/fix-signature

Fix function signature for newer SQLAlchemy

fix function signature for newer SQLAlchemy

Merge pull request #3684 from lonvia/compact-en-variants

Clean up English variants