From 509f59b1930a618eee892303fb483deade1e6dc2 Mon Sep 17 00:00:00 2001 From: James Addison Date: Sun, 1 Feb 2026 18:42:24 +0000 Subject: [PATCH] Refactor: add default params to `index_boundaries` --- src/nominatim_db/indexer/indexer.py | 4 ++-- test/python/indexer/test_indexing.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/nominatim_db/indexer/indexer.py b/src/nominatim_db/indexer/indexer.py index ccbbf117..344fee72 100644 --- a/src/nominatim_db/indexer/indexer.py +++ b/src/nominatim_db/indexer/indexer.py @@ -59,7 +59,7 @@ class Indexer: if await self.index_by_rank(0, 4) > 0: _analyze() - if await self.index_boundaries(0, 30) > 100: + if await self.index_boundaries() > 100: _analyze() if await self.index_by_rank(5, 25) > 100: @@ -74,7 +74,7 @@ class Indexer: if not self.has_pending(): break - async def index_boundaries(self, minrank: int, maxrank: int) -> int: + async def index_boundaries(self, minrank: int = 0, maxrank: int = 30) -> int: """ Index only administrative boundaries within the given rank range. """ total = 0 diff --git a/test/python/indexer/test_indexing.py b/test/python/indexer/test_indexing.py index 54299a96..947403b1 100644 --- a/test/python/indexer/test_indexing.py +++ b/test/python/indexer/test_indexing.py @@ -247,7 +247,7 @@ async def test_index_boundaries(test_db, threads, test_tokenizer): assert test_db.osmline_unindexed() == 1 idx = indexer.Indexer('dbname=test_nominatim_python_unittest', test_tokenizer, threads) - await idx.index_boundaries(0, 30) + await idx.index_boundaries() assert test_db.placex_unindexed() == 31 assert test_db.osmline_unindexed() == 1