fix centcom, resave ruins #1033
Conversation
|
""" WalkthroughВ этом обновлении удалён вызов компонента SharedMoverController в системе BlobPodSystem, что изменяет порядок выполнения метода Zombify: вызов _mover.SetRelay больше не выполняется. Дополнительно внесены изменения в описание сущностей на карте – добавлены новые объекты и переименованы или удалены устаревшие. В файле калькулятора добавлен новый метод, изменена сигнатура существующего и переименована глобальная переменная. Также изменены условия проверки коллизий в методах телепортации и использования психических способностей. Changes
Sequence Diagram(s)sequenceDiagram
participant BP as BlobPodSystem
participant Ent as Сущность
BP->>BP: Инициация процесса zombification
Note right of BP: Вызов movement relay удалён
BP->>Ent: Применение эффектов zombification
Ent-->>BP: Завершение процесса
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (4)
⏰ Context from checks skipped due to timeout of 300000ms (2)
🔇 Additional comments (12)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
S: Untriaged
size/XL
Changes: Map
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Content.Server/Backmen/Blob/NPC/BlobPod/BlobPodSystem.cs(2 hunks)Resources/Maps/Corvax/Ruins/corvax_ussp_asteroid.yml(14 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (1)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (15)
Content.Server/Backmen/Blob/NPC/BlobPod/BlobPodSystem.cs (2)
121-121: Проверьте последствия отключения SetRelay.Отключение _mover.SetRelay может повлиять на синхронизацию движения между BlobPod и целевой сущностью. Необходимо убедиться, что это не приведет к проблемам с управлением зомбифицированными сущностями.
Рекомендуется:
- Документировать причину отключения этой функциональности
- Убедиться, что существует альтернативный механизм контроля движения
38-38: Проверьте влияние удаления SharedMoverController.Удаление зависимости от SharedMoverController может повлиять на движение зомбифицированных сущностей. Убедитесь, что это изменение не нарушает игровую механику.
Resources/Maps/Corvax/Ruins/corvax_ussp_asteroid.yml (13)
12-36: Проверка структуры объектов и иерархии.
В этом блоке определяется основная структура карты: создаётся сущность с uid: 1 и указываются её компоненты, а для сущности с uid: 2 добавлено поле "parent: 1". Убедитесь, что иерархия объектов и родительские связи соответствуют проектным требованиям.
569-582: Проверка списка значений в настройках.
Добавлены дополнительные значения "0" (строки 572–579) в список параметров. Рекомендуется проверить, что длина и значения списка соответствуют спецификации (например, для определения размера сетки или иных расчетов) и, при необходимости, добавить комментарий для разъяснения назначения этих значений.
674-680: Удаление прототипа AirlockHatchBasik.
В строке 677 удалён вызов "proto: AirlockHatchBasik". Проверьте, что данное удаление соответствует общей логике обновления: если новый прототип должен добавляться в другом месте или заменять старый, то данное изменение корректно.
11255-11267: Удаление устаревших прототипов.
В этом блоке удаляются строки с "proto: BackmenVendingMachineSovietSoda" (строка 11258) и "proto: BaseUplinkRadio" (строка 11265). Убедитесь, что соответствующие сущности больше не используются и что их удаление согласуется с общей стратегией обновления объектов карты.
11526-11532: Удаление прототипа CartridgeRifleLight762x39.
Строка 11529 удаляет прототип "CartridgeRifleLight762x39". Необходимо удостовериться, что исключение этого элемента не нарушает работу системы оружия и что его удаление согласовано с изменениями в других связанных сущностях.
11916-11922: Добавление слоя TeleportLayer (хунк 6).
В строке 11919 в список слоёв добавлен элемент "TeleportLayer". Рекомендуется проверить, что это изменение соответствует требованиям физической модели и корректно обрабатывается системой столкновений.
11970-11976: Добавление слоя TeleportLayer (хунк 7).
Аналогично предыдущему блоку, в строке 11973 добавлен "TeleportLayer". Убедитесь, что единообразное применение данного слоя во всех сходных участках не приводит к конфликтам или некорректной обработке физики.
12003-12009: Проверка слоя TeleportLayer (хунк 8).
Добавление "TeleportLayer" (строка 12006) повторяется и здесь. Проверьте, что обновлённая конфигурация слоев согласована с остальными настройками и не вызывает нарушений в расчётах физики объектов на карте.
12036-12042: Унификация настроек слоёв (хунк 9).
В строке 12039 вновь добавлен "TeleportLayer". Рекомендуется удостовериться, что все секции карты используют единый набор правил для обработки слоев, что позволит избежать потенциальных проблем при рендеринге или столкновениях.
13108-13114: Проверка физического слоя при повышенной плотности.
В данном блоке (строка 13111) добавлен "TeleportLayer", при этом плотность установлена в 350. Убедитесь, что такие параметры корректно обрабатываются игровым движком и не приводят к ошибкам в системе столкновений.
13154-13160: Проверка повторного добавления слоя TeleportLayer.
Аналогичным образом, в строках 13156–13160 добавлен "TeleportLayer" для секции с плотностью 350. Рекомендуется убедиться, что подобные повторяющиеся изменения согласованы между собой и корректно влияют на физику объектов.
13896-13902: Удаление прототипа WeaponAKMSRifle.
В строке 13899 происходит удаление вызова "proto: WeaponAKMSRifle". Необходимо проверить, что замена или исключение этого прототипа согласованы с новой схемой оружия и что все ссылки на данный элемент обновлены в соответствии с изменениями.
13908-13914: Удаление прототипа WeaponSniperMosinNew.
Строка 13911 удаляет вызов "proto: WeaponSniperMosinNew". Проверьте, что новая номенклатура или замена оружия реализованы последовательно по всему проекту, и что удаление этого прототипа не нарушает логику работы системы вооружения.
| pos: 13.5,-8.5 | ||
| parent: 2 | ||
| - type: Door | ||
| secondsUntilStateChange: -14372.478 | ||
| secondsUntilStateChange: -14393.855 | ||
| state: Opening | ||
| - uid: 151 | ||
| components: |
There was a problem hiding this comment.
Некорректное значение времени переключения состояния.
Параметр "secondsUntilStateChange" имеет значение -14393.855, что выглядит некорректно – обычно ожидается положительное число. Проверьте алгоритм расчёта и логику работы дверей, так как отрицательное значение может вызвать неэффективное или неверное поведение.
🆑
Summary by CodeRabbit