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

Add "Polyhedra Wireframe" feature from Tissue addon as a scripted node.

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние polyhedra_snl в master Дек 28, 2020
  • Обзор 1
  • Коммиты 1
  • Конвейеры 0
  • Изменения 1

Created by: portnov

I wanted to have this feature in Sverchok long ago... Unfortunately, at the moment, this feature in Tissue has the following problems:

  • it is not defined as a library, instead the whole code is all inside one operator;
  • I'm not sure if we can effectively use Tissue addon as a dependency;
  • the code does not work for some corner cases, giving exceptions or weird geometry;
  • code style is not particularly good, and the code is not very well documented, so I, for once, cann't be sure how to fix bugs;
  • I have some general ideas how to make it's performance better, but again, I can't implement it in this code in it's current state.

So, with current state of the code, I don't think it is a good idea to either use Tissue as dependency or to copy-paste the code (effectively creating a fork).

So, as some temporary solution, I'm adding a copy-paste of this code as a scripted node example.

Screenshot_20201228_222529

Screenshot_20201228_223559

Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: polyhedra_snl