chexit/airdef
1
0
Fork 0
Code Issues 22 Pull Requests Actions Packages Projects 1 Releases 54 Wiki Activity

[bug] Occlusion свечения сравнивает осевую глубину с расстоянием вдоль луча #14

New Issue
Closed
opened 2026-10-29 11:59:24 +00:00 by chexit · 1 comment
chexit commented 2026-10-29 11:59:24 +00:00
Owner
Copy Link
Copy Source

Проверка исходников на 90379f6. Полный Horizon/GPU на Mac не запускался; оценки сложности и ресурсов ниже не являются замерами FPS.

В prelude gfx/mod_post.rs: project(world).z возвращает clip c.w (осевую глубину для perspective projection). scene_distance(uv) возвращает length(view_ray_point(...)) — расстояние вдоль луча. effects.wgsl сравнивает эти разные величины для visibility вспышек и отсечения glow.

Математический контрпример, без запуска GPU: точка в координатах камеры (10, 0, -10) имеет осевую глубину 10 и расстояние sqrt(200)≈14.142. Стена на том же луче на расстоянии 12 закрывает точку, но сравнение 12 < 10 ложно. Чем дальше от центра экрана, тем сильнее расхождение. Для фактических sphere radii выбрать радиус существенно меньше этой разницы.

Предложение: сохранить c.w для определения знака/положения относительно камеры, а при occlusion использовать одну метрику на обеих сторонах (radial distance либо обе axial depths) и согласованные радиусные допуски. Не менять project API молча, если есть другие consumers.

Критерий: тест с источником за стеной в центре и у края экрана; корректная видимость при повороте камеры и изменении FOV, без просвечивания glow через препятствие. Нужен GPU image regression для окончательной визуальной проверки.

Код:

airdef/horizon-patches/0003-Mod-API-AirDef-fork-full-screen-post-pass-for-mods.patch Line 262 in 90379f6
+fn project(world: vec3<f32>) -> vec3<f32> {
(prelude),
airdef/airdef-mod/shaders/effects.wgsl Line 200 in 90379f6
let visible = select(0.25, 1.0, scene_distance(p.xy) > p.z - a.w);
,
airdef/airdef-mod/shaders/effects.wgsl Line 214 in 90379f6
if scene_distance(p.xy) < p.z - a.w * 1.5 { continue; }
.

Проверка исходников на `90379f6`. Полный Horizon/GPU на Mac не запускался; оценки сложности и ресурсов ниже не являются замерами FPS. В prelude `gfx/mod_post.rs`: `project(world).z` возвращает clip `c.w` (осевую глубину для perspective projection). `scene_distance(uv)` возвращает `length(view_ray_point(...))` — расстояние вдоль луча. `effects.wgsl` сравнивает эти разные величины для visibility вспышек и отсечения glow. Математический контрпример, без запуска GPU: точка в координатах камеры (10, 0, -10) имеет осевую глубину 10 и расстояние sqrt(200)≈14.142. Стена на том же луче на расстоянии 12 закрывает точку, но сравнение `12 < 10` ложно. Чем дальше от центра экрана, тем сильнее расхождение. Для фактических sphere radii выбрать радиус существенно меньше этой разницы. Предложение: сохранить c.w для определения знака/положения относительно камеры, а при occlusion использовать одну метрику на обеих сторонах (radial distance либо обе axial depths) и согласованные радиусные допуски. Не менять project API молча, если есть другие consumers. Критерий: тест с источником за стеной в центре и у края экрана; корректная видимость при повороте камеры и изменении FOV, без просвечивания glow через препятствие. Нужен GPU image regression для окончательной визуальной проверки. Код: https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/horizon-patches/0003-Mod-API-AirDef-fork-full-screen-post-pass-for-mods.patch#L262 (prelude), https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/airdef-mod/shaders/effects.wgsl#L200, https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/airdef-mod/shaders/effects.wgsl#L214.
chexit added the area:renderingpriority:P2type:bug labels 2026-10-29 13:09:47 +00:00
chexit commented 2026-10-29 13:29:21 +00:00
Author
Owner
Copy Link
Copy Source

🤖 Agent: исправлено в https://gitea.maoniao.team/chexit/airdef/commit/67a2a52 (запушено в main).

🤖 Agent: исправлено в https://gitea.maoniao.team/chexit/airdef/commit/67a2a52 (запушено в main).
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area:architecture
Границы модулей, контракты, состояние сессии и развитие общей архитектуры
 area:audio
Звуки, громкость, задержка распространения и аудиоассеты.
 area:logistics
Боезапас, склады, ТЗМ, обслуживание и заявки снабжения.
 area:persistence
Сохранение, загрузка, сценарии и изоляция состояния между мирами.
 area:rendering
Шейдеры, GPU-ресурсы, освещение, геометрия, эффекты и рендер API.
 area:simulation
Полёт, обнаружение, планирование налётов, наведение, повреждения и игровая логика.
 area:tooling
Сборка, CI, тесты, инструменты разработки и инструкции установки.
 area:ui
HUD, радары, мониторы, меню, текст и интерфейс реплеев.
 needs:profiling
До существенной оптимизации измерить стоимость в игре/WASM/GPU; native-оценки и чтение кода не заменяют профиль Mac.
 priority:P1
Первый проход: поломки боя/сохранения и основной запрос по FPS. Гипотезы производительности сначала профилировать.
 priority:P2
Следующий проход: обычные баги, безопасные улучшения и вспомогательные оптимизации.
 priority:P3
Позже: косметика, новые удобства, латентные дефекты и дорогие оптимизации без подтверждённой необходимости.
 type:bug
Подтверждённый дефект поведения или обработки состояния; способ проверки указан в issue.
 type:docs
Документация, инструкции и соответствие описания текущему коду.
 type:enhancement
Новое поведение, баланс или улучшение удобства.
 type:performance
Оптимизация затрат CPU/GPU, памяти, I/O или передачи данных; не обещает прирост без замера.
No Label area:rendering priority:P2 type:bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
chexit
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: chexit/airdef#14
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?