Skip to content

Ambience refactor, sound caching and TRAIT_DEAF implementation#9130

Open
littleboobs wants to merge 40 commits intoss220-space:master220from
littleboobs:ambience
Open

Ambience refactor, sound caching and TRAIT_DEAF implementation#9130
littleboobs wants to merge 40 commits intoss220-space:master220from
littleboobs:ambience

Conversation

@littleboobs
Copy link
Copy Markdown
Collaborator

@littleboobs littleboobs commented May 3, 2026

Что этот ПР делает

Список изменений

🆑
refactor: Система эмбиенса и белого шума.
refactor: Кеширование звуков.
sound: Очистка звуков и новые звуки белого шума и эмбиенса.
bugfix: Исправлена работа микшера громкости.
/:cl:

@littleboobs littleboobs requested a review from Bizzonium as a code owner May 3, 2026 16:34
@github-actions github-actions Bot added TGUI PR содержит изменения в файлах TGUI. Звук PR изменяет звуковые эффекты. Refactor PR проводит рефакторинг кода. Fix PR содержит исправления ошибок. labels May 3, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена. Данный пулл-реквест проводит масштабную реорганизацию системы эмбиенса, внедряя глобальные списки звуков и механизм кеширования их длительности через rust_g, а также переводит проверку слуха на систему трейтов. В ходе ревью были обнаружены критические ошибки: риск рантайма при пустом списке звуков в play_ambience и некорректная передача списка вместо строки в ambience_index. Также выявлены логические несоответствия в работе принудительного эмбиенса, избыточность системных уведомлений в общем чате и нарушения стайлгайда, касающиеся производительности циклов и кеширования переменных датумов. Требуется корректировка кулдаунов и исправление логики итераций в коде кресел-каталок.

Comment thread code/controllers/subsystem/ambience.dm
Comment thread code/game/area/ss13_areas.dm Outdated
Comment thread code/controllers/subsystem/ambience.dm Outdated
Comment thread code/controllers/subsystem/ambience.dm
Comment thread code/controllers/subsystem/ambience.dm
Comment thread code/controllers/subsystem/non-firing/sounds.dm Outdated
Comment thread code/game/area/areas.dm
Comment thread code/modules/vehicle/wheelchair.dm Outdated
@littleboobs littleboobs requested a review from a team as a code owner May 3, 2026 17:14
@github-actions github-actions Bot added the Изменение карты PR содержит изменения в файлах карт. label May 3, 2026
@github-actions github-actions Bot added the Rust PR содержит изменения в файлах Rust кода. label May 3, 2026
@dageavtobusnick
Copy link
Copy Markdown
Collaborator

!build_rust

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2026

Building Rust library failed, see the action run log for details: https://github.com/ss220-space/Paradise/actions/runs/25287974307

@dageavtobusnick
Copy link
Copy Markdown
Collaborator

!build_rust

@github-actions github-actions Bot added the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label May 4, 2026
@littleboobs
Copy link
Copy Markdown
Collaborator Author

!build_rust

@littleboobs littleboobs added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels May 4, 2026
Comment thread code/modules/vehicle/wheelchair.dm Outdated
Comment thread code/controllers/subsystem/non-firing/sounds.dm Outdated
Comment thread code/controllers/subsystem/non-firing/sounds.dm Outdated
Comment thread code/controllers/subsystem/non-firing/sounds.dm Outdated
Comment thread code/controllers/subsystem/ambience.dm Outdated
Comment thread code/controllers/subsystem/ambience.dm Outdated
@dageavtobusnick dageavtobusnick added Testmerge PR в данный момент проходит тестирование на серверах. Testmerge Candidate PR скоро будет в тестмёрже. labels May 5, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels May 5, 2026
@dageavtobusnick
Copy link
Copy Markdown
Collaborator

!build_rust

@github-actions github-actions Bot removed the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label May 5, 2026
github-actions Bot pushed a commit that referenced this pull request May 5, 2026
github-actions Bot pushed a commit that referenced this pull request May 5, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels May 6, 2026
github-actions Bot pushed a commit that referenced this pull request May 6, 2026
github-actions Bot pushed a commit that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix PR содержит исправления ошибок. Refactor PR проводит рефакторинг кода. Rust PR содержит изменения в файлах Rust кода. Testmerge Candidate PR скоро будет в тестмёрже. Testmerge PR в данный момент проходит тестирование на серверах. TGUI PR содержит изменения в файлах TGUI. Звук PR изменяет звуковые эффекты. Изменение карты PR содержит изменения в файлах карт. Инструменты PR вносит изменения в инструменты разработки, конфигурации IDE или конфигурацию AI помощника.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants