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

Adding inputs lengths matching and solving a few minor issues/limitations of Mask Join node

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние github/fork/ArpegorPSGH/mask_join_add_global_match в mask_join_match Авг 20, 2022
  • Обзор 56
  • Коммиты 12
  • Конвейеры 0
  • Изменения 8

Created by: ArpegorPSGH

Addressed problem description

This solves issue #4607 (closed). I also solved what looked like an issue and slightly expanded the node functionality.

Solution description

I added list length matching at each level in the inputs, while not changing the node treatment on the lists. In the get_level function, it seemed to me that the case >2 should instead be >= 2, otherwise the treatment is not consistent when you increase the number of levels. Finally, I enabled the use of the choice option when there is 0 nesting in the data input lists.

Preflight checklist

  • Code changes complete.
  • Code documentation complete.
  • Documentation for users complete (or not required, if user never sees these changes).
  • Manual testing done.
  • Unit-tests implemented.
  • Ready for merge.
Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/ArpegorPSGH/mask_join_add_global_match