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

Dictionary in and dictionary out nodes

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние dictionary_socket в b28_prelease_master Дек 28, 2019
  • Обзор 44
  • Коммиты 13
  • Конвейеры 0
  • Изменения 9

Created by: Durman

Addressed problem description

#2766 (closed)

image

Todo:

  • Take default name of key from linked socket.
  • Keep links of dictionary out nodes when a key of a dictionary was renamed. Next time or probably it even does not worth complicate too much. Simplest solutions was adding custom attribute for dictionaries.
  • Robustness of determining type of output sockets. Next time. Simplest solutions was adding custom attribute for dictionaries.
  • Consider handling of several dictionaries in list (ignoring mismatched keys?).
  • Сustom draw for all sockets.
  • Is_linked should have name was_linked before start of update event, more correct test is socket.links but as I suppose more expensive as well. Looks like in sv_get method is_linked should be replaced by links.
  • deepcopy = False where it possible Deep copy is not implemented for dictionaries. In next PR.
  • New dictionary icons 😄

Preflight checklist

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