Skip to content

По умолчанию SymmetrySplit только для source LOD; опциональный режим для target LOD с предупреждением - #16

Merged
SashaRX merged 1 commit into
masterfrom
codex/refactor-execsymmetrysplit-to-handle-source-lod
Mar 3, 2026
Merged

По умолчанию SymmetrySplit только для source LOD; опциональный режим для target LOD с предупреждением#16
SashaRX merged 1 commit into
masterfrom
codex/refactor-execsymmetrysplit-to-handle-source-lod

Conversation

@SashaRX

@SashaRX SashaRX commented Mar 3, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Предотвратить непреднамеренную модификацию топологии целевых LOD при автоматическом запуске SymmetrySplit в Full Pipeline, чтобы сохранить стабильность последующих шагов Transfer/Apply.
  • Дать пользователю явный контроль над применением SymmetrySplit к target LOD и предупредить о рисках изменения топологии.

Description

  • Параметризовал ExecSymmetrySplit(bool includeTargets) и по умолчанию вызываю его только для sourceLodIndex, а split по target LOD доступен через новый флаг UI splitTargetsInSymmetryStep.
  • Добавил чекбокс SymSplit target LODs (advanced / risky) и HelpBox с предупреждением в интерфейс, а также обновлённую подпись pipeline-строки чтобы явно показывать режим (source only / source + targets).
  • Внёс улучшенные логи в шаг SymmetrySplit: раздельный подсчёт split-ов по source/targets, список модифицированных LOD и предупреждение при изменении топологии target LOD.
  • Добавил диагностику в downstream: в ExecTransferAll() логируется какой LOD используется как источник и сколько mesh были repacked, а также предупреждение если target LOD были ранее модифицированы; в ExecTransferLod() добавлена проверка на рассинхрон vertexCount между предыдущим transferredMesh и текущим target mesh.

Testing

  • Выполнены статические проверки формата с git diff --check, проблем не обнаружено.
  • Проверен рабочий diff файла Editor/UvTransferWindow.cs и просмотр вставленных логов/UI-вызовов в исходнике.
  • В этой среде не выполнялся Unity Editor / PlayMode тест-раннер, поэтому интеграционные проверки Transfer/Apply в Unity не запускались.

Codex Task

SashaRX pushed a commit that referenced this pull request Mar 3, 2026
…urce-only split)

Merged PRs:
- #18 codex/update-shell-computation-logic (majority vote)
- #17 codex/add-symmetry-split-support-to-sidecar-metadata (sidecar flag)
- #16 codex/refactor-execsymmetrysplit-to-handle-source-lod (source-only default)

Resolved merge conflict between PR #17 and #16 in ExecSymmetrySplit():
combined LOD filtering with wasSymmetrySplit tracking.

Removed REVIEW.md (deleted by all 3 PRs).

https://claude.ai/code/session_01198JqNgkdbzwRFc7nt8FqJ
@SashaRX
SashaRX merged commit 126e3f5 into master Mar 3, 2026
@SashaRX
SashaRX deleted the codex/refactor-execsymmetrysplit-to-handle-source-lod branch March 4, 2026 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant