X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/9659afbade47d1ec6d5359b2b21e1e874516ed80..1f0796778754d8df0dfab9dd01302e26a397f064:/src/nominatim_db/clicmd/setup.py?ds=sidebyside diff --git a/src/nominatim_db/clicmd/setup.py b/src/nominatim_db/clicmd/setup.py index 07a76f59..39cbe65a 100644 --- a/src/nominatim_db/clicmd/setup.py +++ b/src/nominatim_db/clicmd/setup.py @@ -23,13 +23,10 @@ from ..tokenizer.base import AbstractTokenizer from ..version import NOMINATIM_VERSION from .args import NominatimArgs -# Do not repeat documentation of subcommand classes. -# pylint: disable=C0111 -# Using non-top-level imports to avoid eventually unused imports. -# pylint: disable=C0415 LOG = logging.getLogger() + class SetupAll: """\ Create a new Nominatim database from an OSM file. @@ -42,36 +39,35 @@ class SetupAll: def add_args(self, parser: argparse.ArgumentParser) -> None: group1 = parser.add_argument_group('Required arguments') group1.add_argument('--osm-file', metavar='FILE', action='append', - help='OSM file to be imported' - ' (repeat for importing multiple files)', - default=None) + help='OSM file to be imported' + ' (repeat for importing multiple files)', + default=None) group1.add_argument('--continue', dest='continue_at', - choices=['import-from-file', 'load-data', 'indexing', 'db-postprocess'], - help='Continue an import that was interrupted', - default=None) + choices=['import-from-file', 'load-data', 'indexing', 'db-postprocess'], + help='Continue an import that was interrupted', + default=None) group2 = parser.add_argument_group('Optional arguments') group2.add_argument('--osm2pgsql-cache', metavar='SIZE', type=int, - help='Size of cache to be used by osm2pgsql (in MB)') + help='Size of cache to be used by osm2pgsql (in MB)') group2.add_argument('--reverse-only', action='store_true', - help='Do not create tables and indexes for searching') + help='Do not create tables and indexes for searching') group2.add_argument('--no-partitions', action='store_true', - help=("Do not partition search indices " - "(speeds up import of single country extracts)")) + help="Do not partition search indices " + "(speeds up import of single country extracts)") group2.add_argument('--no-updates', action='store_true', - help="Do not keep tables that are only needed for " - "updating the database later") + help="Do not keep tables that are only needed for " + "updating the database later") group2.add_argument('--offline', action='store_true', help="Do not attempt to load any additional data from the internet") group3 = parser.add_argument_group('Expert options') group3.add_argument('--ignore-errors', action='store_true', - help='Continue import even when errors in SQL are present') + help='Continue import even when errors in SQL are present') group3.add_argument('--index-noanalyse', action='store_true', - help='Do not perform analyse operations during index (expert only)') + help='Do not perform analyse operations during index (expert only)') group3.add_argument('--prepare-database', action='store_true', help='Create the database but do not import any data') - - def run(self, args: NominatimArgs) -> int: # pylint: disable=too-many-statements, too-many-branches + def run(self, args: NominatimArgs) -> int: if args.osm_file is None and args.continue_at is None and not args.prepare_database: raise UsageError("No input files (use --osm-file).") @@ -85,10 +81,9 @@ class SetupAll: return asyncio.run(self.async_run(args)) - async def async_run(self, args: NominatimArgs) -> int: from ..data import country_info - from ..tools import database_import, refresh, postcodes, freeze + from ..tools import database_import, postcodes, freeze from ..indexer.indexer import Indexer num_threads = args.threads or psutil.cpu_count() or 1 @@ -97,7 +92,7 @@ class SetupAll: if args.prepare_database or args.continue_at is None: LOG.warning('Creating database') database_import.setup_database_skeleton(args.config.get_libpq_dsn(), - rouser=args.config.DATABASE_WEBUSER) + rouser=args.config.DATABASE_WEBUSER) if args.prepare_database: return 0 @@ -120,8 +115,7 @@ class SetupAll: postcodes.update_postcodes(args.config.get_libpq_dsn(), args.project_dir, tokenizer) - if args.continue_at in \ - ('import-from-file', 'load-data', 'indexing', None): + if args.continue_at in ('import-from-file', 'load-data', 'indexing', None): LOG.warning('Indexing places') indexer = Indexer(args.config.get_libpq_dsn(), tokenizer, num_threads) await indexer.index_full(analyse=not args.index_noanalyse) @@ -141,16 +135,10 @@ class SetupAll: LOG.warning('Recompute word counts') tokenizer.update_statistics(args.config, threads=num_threads) - webdir = args.project_dir / 'website' - LOG.warning('Setup website at %s', webdir) - with connect(args.config.get_libpq_dsn()) as conn: - refresh.setup_website(webdir, args.config, conn) - self._finalize_database(args.config.get_libpq_dsn(), args.offline) return 0 - def _base_import(self, args: NominatimArgs) -> None: from ..tools import database_import, refresh from ..data import country_info @@ -164,8 +152,8 @@ class SetupAll: database_import.check_existing_database_plugins(args.config.get_libpq_dsn()) LOG.warning('Setting up country tables') country_info.setup_country_tables(args.config.get_libpq_dsn(), - args.config.lib_dir.data, - args.no_partitions) + args.config.lib_dir.data, + args.no_partitions) LOG.warning('Importing OSM data file') database_import.import_osm_data(files, @@ -176,20 +164,19 @@ class SetupAll: LOG.warning('Importing wikipedia importance data') data_path = Path(args.config.WIKIPEDIA_DATA_PATH or args.project_dir) if refresh.import_wikipedia_articles(args.config.get_libpq_dsn(), - data_path) > 0: + data_path) > 0: LOG.error('Wikipedia importance dump file not found. ' - 'Calculating importance values of locations will not ' - 'use Wikipedia importance data.') + 'Calculating importance values of locations will not ' + 'use Wikipedia importance data.') LOG.warning('Importing secondary importance raster data') if refresh.import_secondary_importance(args.config.get_libpq_dsn(), - args.project_dir) != 0: + args.project_dir) != 0: LOG.error('Secondary importance file not imported. ' - 'Falling back to default ranking.') + 'Falling back to default ranking.') self._setup_tables(args.config, args.reverse_only) - def _setup_tables(self, config: Configuration, reverse_only: bool) -> None: """ Set up the basic database layout: tables, indexes and functions. """ @@ -210,7 +197,6 @@ class SetupAll: LOG.warning('Create functions (3rd pass)') refresh.create_functions(conn, config, False, False) - def _get_tokenizer(self, continue_at: Optional[str], config: Configuration) -> AbstractTokenizer: """ Set up a new tokenizer or load an already initialised one. @@ -224,7 +210,6 @@ class SetupAll: # just load the tokenizer return tokenizer_factory.get_tokenizer_for_db(config) - def _finalize_database(self, dsn: str, offline: bool) -> None: """ Determine the database date and set the status accordingly. """ @@ -235,5 +220,5 @@ class SetupAll: dbdate = status.compute_database_date(conn, offline) status.set_status(conn, dbdate) LOG.info('Database is at %s.', dbdate) - except Exception as exc: # pylint: disable=broad-except + except Exception as exc: LOG.error('Cannot determine date of database: %s', exc)