(medium) 12_more_pois.py/main: bbox padding ±0.005° меньше радиуса привязки 2 км — POI у границ теряются #1510

Closed
opened 2026-06-15 05:58:58 +00:00 by bot-backend · 1 comment
Collaborator

Severity: medium · Категория: data-integrity · Файл: site-finder/12_more_pois.py:55-55

12_more_pois.py:55 — b = (min(lats)-0.005, ..., max(lats)+0.005), т.е. паддинг ±0.005° (~0.5 км). При этом привязка POI к участку идёт в радиусе 2 км (строка 89: d<=2000). Для участков у границы общего bbox реально существующие POI на расстоянии 0.5–2 км за пределами bbox не попадают в выгрузку Overpass → distance_m/счётчики для новых категорий (parking, gym, cinema и т.д.) занижены/пусты. Соседние скрипты 02_fetch_pois.py:63 и 11_refetch_pois_extended.py:67 используют ±0.05° (~5.5 км), что согласовано с 2-км радиусом. Импакт: заниженные метрики для приграничных участков. Фикс: заменить 0.005 на 0.05 (вероятная опечатка).

Почему баг: Радиус привязки POI к участку — 2 км (d<=2000 в цикле), а паддинг bbox всего 0.5 км. Для приграничных участков реально существующие POI в зоне 0.5–2 км вне bbox отсутствуют в d.elements → теряются данные. Несогласованность с ±0.05° в соседних скриптах подтверждает, что это опечатка (0.005 вместо 0.05).

Как чинить: Использовать тот же паддинг ≥0.05° (≈5.5 км) как в 02/11, чтобы покрыть радиус привязки 2 км для всех участков.


deep-audit «армия ботов 2» · verify+harden, confidence 0.88 · unit U114

**Severity:** medium · **Категория:** data-integrity · **Файл:** `site-finder/12_more_pois.py:55-55` 12_more_pois.py:55 — b = (min(lats)-0.005, ..., max(lats)+0.005), т.е. паддинг ±0.005° (~0.5 км). При этом привязка POI к участку идёт в радиусе 2 км (строка 89: d<=2000). Для участков у границы общего bbox реально существующие POI на расстоянии 0.5–2 км за пределами bbox не попадают в выгрузку Overpass → distance_m/счётчики для новых категорий (parking, gym, cinema и т.д.) занижены/пусты. Соседние скрипты 02_fetch_pois.py:63 и 11_refetch_pois_extended.py:67 используют ±0.05° (~5.5 км), что согласовано с 2-км радиусом. Импакт: заниженные метрики для приграничных участков. Фикс: заменить 0.005 на 0.05 (вероятная опечатка). **Почему баг:** Радиус привязки POI к участку — 2 км (d<=2000 в цикле), а паддинг bbox всего 0.5 км. Для приграничных участков реально существующие POI в зоне 0.5–2 км вне bbox отсутствуют в d.elements → теряются данные. Несогласованность с ±0.05° в соседних скриптах подтверждает, что это опечатка (0.005 вместо 0.05). **Как чинить:** Использовать тот же паддинг ≥0.05° (≈5.5 км) как в 02/11, чтобы покрыть радиус привязки 2 км для всех участков. --- <sub>deep-audit «армия ботов 2» · verify+harden, confidence 0.88 · unit U114</sub>
bot-backend added the
week ревью 1
label 2026-06-15 05:58:58 +00:00
bot-backend added the
scope/backend
label 2026-06-15 11:02:16 +00:00
Owner

🔍 week ревью 1 — верификация (ПОДТВЕРЖДЕНО)

Текущее состояние кода: site-finder/12_more_pois.py:55 (bbox padding), :89 (радиус привязки).

Корень: :55 b = (min(lats)-0.005, min(lons)-0.005, max(lats)+0.005, max(lons)+0.005) — паддинг ±0.005° (~0.5 км). Привязка POI к участку — в радиусе 2 км (:89 if d <= 2000). Для участков у границы общего bbox реально существующие POI на 0.5–2 км за пределами bbox не попадают в выгрузку Overpass (:63 overpass(filt, b)) → distance_m/счётчики для новых категорий (parking, gym, cinema и т.д.) занижены/пусты. Соседние скрипты используют ±0.05°: 02_fetch_pois.py:63 (pad_deg=0.05) и 11_refetch_pois_extended.py:67 (-0.05) — согласовано с 2-км радиусом. Подтверждённая опечатка (0.005 вместо 0.05).

Что править:

  • Заменить 0.005 на 0.05 во всех четырёх компонентах b на :55 (≈5.5 км паддинг покрывает 2-км радиус для всех приграничных участков), как в 02/11.

Что посмотреть:

  • Код: 12_more_pois.py:55,89; 02_fetch_pois.py:63; 11_refetch_pois_extended.py:67.
  • Vault: релевантных заметок не найдено.
**🔍 week ревью 1 — верификация (ПОДТВЕРЖДЕНО)** **Текущее состояние кода:** `site-finder/12_more_pois.py:55` (bbox padding), `:89` (радиус привязки). **Корень:** `:55` `b = (min(lats)-0.005, min(lons)-0.005, max(lats)+0.005, max(lons)+0.005)` — паддинг ±0.005° (~0.5 км). Привязка POI к участку — в радиусе 2 км (`:89` `if d <= 2000`). Для участков у границы общего bbox реально существующие POI на 0.5–2 км за пределами bbox не попадают в выгрузку Overpass (`:63 overpass(filt, b)`) → distance_m/счётчики для новых категорий (parking, gym, cinema и т.д.) занижены/пусты. Соседние скрипты используют ±0.05°: `02_fetch_pois.py:63` (`pad_deg=0.05`) и `11_refetch_pois_extended.py:67` (`-0.05`) — согласовано с 2-км радиусом. Подтверждённая опечатка (0.005 вместо 0.05). **Что править:** - Заменить `0.005` на `0.05` во всех четырёх компонентах `b` на `:55` (≈5.5 км паддинг покрывает 2-км радиус для всех приграничных участков), как в 02/11. **Что посмотреть:** - Код: `12_more_pois.py:55,89`; `02_fetch_pois.py:63`; `11_refetch_pois_extended.py:67`. - Vault: релевантных заметок не найдено.
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#1510
No description provided.