search

rendrom / rosreestr2coord

Вычисление координат участка по кадастровому номеру с сайта https://pkk.rosreestr.ru
MIT License
167 stars 79 forks source link

Refactor/download feature info #74

Closed NiKuma0 closed 1 year ago

NiKuma0 commented 1 year ago

Небольшой рефактор кода.

Не понимаю, зачем тут использовался try, except с методом self.error? И без него все ошибки падают в логгер.

rendrom commented 1 year ago

Почму ушёл TimeoutException ?

NiKuma0 commented 1 year ago

Почму ушёл TimeoutException ?

С TimeoutException-ом вообще странно - получается если он падает, то его не надо логировать, но надо зарайзить из-за чего он падает в логер. В чем смысл? Очевидно это работает не правильно...

Тут нужно настроить логер (logging) и убрать все подобные try, exception из кода + убрать методы логирования у класса Area и использовать вместо них встроенный логгер

rendrom commented 1 year ago

Приношу извинения с затянушимся рассмотрением вашего PR. Но так как в коммитах нет никаких критических фиксов, добавлю изменения в следующую версию с остальными доработками.