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

Bugfix to the formula node

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние github/fork/matteolegna/patch-1 в master Апр 18, 2020
  • Обзор 3
  • Коммиты 1
  • Конвейеры 0
  • Изменения 1

Created by: matteolegna

Hi! I have spotted what I think is a bug. I am quite of a noob; hopefully, I am doing the right thing creating this pull request...

Addressed problem description

I've noticed that the Formula Node does not work when inputting an expression that lacks any variable. For example, typing x + 3 (given the default value for x of 0), it does correctly evaluate outputting 3. Typing 3 + 1 gives no result, as the for loop https://github.com/nortikin/sverchok/blob/e9cfe0c77f07844564a9121ba5821cad96701edf/nodes/number/formula3.py#L190 never runs if var_names == false

Solution description

zip_long_repeat(*objects) returns an empty iterator if objects is ([],). My proposed fix is aimed to make objects equal to ([None],) when there are no variables in the expression. Thus list(zip_long_repeat(*objects)) = (None,) and the for loop runs once.

Preflight checklist

Put an x letter in each brackets when you're done this item:

  • Code changes complete.
  • Manual testing done.
  • Ready for merge.
Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/matteolegna/patch-1