• S sverchok
  • Информация о проекте
    • Информация о проекте
    • Активность
    • Метки
    • Участники
  • Репозиторий
    • Репозиторий
    • Файлы
    • Коммиты
    • Ветки
    • Теги
    • Участники
    • Диаграмма
    • Сравнение
  • Задачи 148
    • Задачи 148
    • Список
    • Доски
    • Спринты
  • Запросы на слияние 21
    • Запросы на слияние 21
  • CI/CD
    • CI/CD
    • Конвейеры
    • Задания
    • Расписания
  • Развертывания
    • Развертывания
    • Окружения
    • Релизы
  • Пакеты и реестры
    • Пакеты и реестры
    • Реестр пакетов
    • Реестр контейнеров
  • Мониторинг
    • Мониторинг
    • Инциденты
  • Аналитика
    • Аналитика
    • Поток ценности
    • CI/CD
    • Репозиторий
  • Wiki
    • Wiki
  • Сниппеты
    • Сниппеты
  • Активность
  • Диаграмма
  • Создать новую задачу
  • Задания
  • Коммиты
  • Доски с задачами
Свернуть панель
  • nikitronn
  • sverchok
  • Запросы на слияние
  • !4891

Logging refactoring

  • Ревью изменений

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние logging_refactoring в master Фев 28, 2023
  • Обзор 3
  • Коммиты 7
  • Конвейеры 0
  • Изменения 141

Created by: Durman

Fixing problems:

  • Logging call was in some another module, not where actual logging was happening. It made such record attributes as modulename or lineno useless.
  • Root logger was configured what could conflict with using logging in other add-ons.

New features

  • Colored logging =) image

How to use logging now?

Inside nodes nothing has changed self.debug(message)

In other modules:

- from sverchok.utils.logging import debug
- debug(message)

# option 1
+ import logging
+ logger = logging.geLogger('sverchok')
+ logger.debug(message)

# option 2
+ from sverchok.utils.sv_logging import sv_logger
+ sv_logger.debug(message)

# option 3
+ from sverchok.utils.sv_logging import get_logger
+ logger = get_logger()
+ logger.debug(message)

For Sverchok extensions first option is especially useful since during reloading event sverchok can reload after an extension. It has sense to add its own sub logger logging.getLogger('sverchok.megapolis')

Preflight checklist

  • Code changes complete.
  • Manual testing done.
Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: logging_refactoring