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

Partially remove limitations of the Cut Object by Surface node

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты nikitronn запросил слияние github/fork/ArpegorPSGH/reduce_scope_limitation_cut_object_by_surface в master Авг 27, 2022
  • Обзор 22
  • Коммиты 7
  • Конвейеры 0
  • Изменения 2

Created by: ArpegorPSGH

Addressed problem description

Solve part of the implementation limitations, as discussed here #4594.

Solution description

I removed the double projection with raycast, removed the limitation when only looking for intersection points, added an option to speed up bvh tree generation when the surface mesh is made of triangles only, and provided an idea on how to completely remove the limitation, along with how to deal with a potential problem with how the intersection points are detected, if it ever arose. See the code comments for more details.

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/reduce_scope_limitation_cut_object_by_surface