IPC radiation and barotrauma remove#983
IPC radiation and barotrauma remove#983Lachklen wants to merge 4 commits intoWWhiteDreamProject:masterfrom
Conversation
WalkthroughPR удаляет или комментирует упоминания damageContainer Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
Resources/Prototypes/Traits/physical.yml (1)
1289-1300: Исправьте функциональность трейта, включив отключённый модификатор.Трейт
SuperiorRadiationShielding(строки 1289-1300) ссылается наdamageModifierSets: SuperiorRadiationShielding, который закомментирован вResources/Prototypes/Damage/modifier_sets.yml(строки 413-417). В результате трейт не будет функционировать, несмотря на наличие описания в локализации.
🧹 Nitpick comments (3)
Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/medical-responsory.yml (1)
179-179: Добавление дыхательной маски не связано с основной целью PR.Добавление
ClothingMaskBreathв содержимое хранилища костюма выглядит как восстановление ранее удаленного предмета, но не связано напрямую с удалением урона от радиации и давления для КПБ.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/security-responsory.yml (1)
179-179: Добавление дыхательной маски не связано с основной целью PR.Как и в medical-responsory.yml, добавление
ClothingMaskBreathвыглядит как несвязанное изменение.Resources/Prototypes/Damage/modifier_sets.yml (1)
423-423: Опциональная очистка форматирования.Добавлена лишняя пустая строка после коэффициентов
HardenedCircuits. Это не влияет на функциональность, но может быть удалено для консистентности форматирования.
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (21)
Resources/Prototypes/Damage/containers.ymlResources/Prototypes/Damage/modifier_sets.ymlResources/Prototypes/Entities/Clothing/Eyes/hud.ymlResources/Prototypes/Entities/Mobs/NPCs/plantbot.ymlResources/Prototypes/Entities/Mobs/NPCs/weldbot.ymlResources/Prototypes/Entities/Mobs/Player/ipc.ymlResources/Prototypes/Entities/Objects/Materials/Sheets/glass.ymlResources/Prototypes/Entities/Objects/Tools/cable_coils.ymlResources/Prototypes/Entities/Objects/Tools/welders.ymlResources/Prototypes/Traits/physical.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/apocryphal.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/corporate.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/engineering-responsory.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/inquisitory.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/janitorial-responsory.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/leader-responsory.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/medical-responsory.ymlResources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/security-responsory.ymlResources/Prototypes/_Goobstation/Entities/Clothing/Eyes/glasses.ymlResources/Prototypes/_NF/Entities/Clothing/Eyes/punk_glasses.ymlResources/Prototypes/_Nuclear14/Entities/Objects/Materials/materials.yml
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/en-US/_white/cards/hobby.ftl:5-5
Timestamp: 2025-10-22T20:13:31.389Z
Learning: В PR по EvacPod для репозитория WWhiteDreamProject/wwdpublic: если опечатка в ключе локализации (например, card-ep_hobby_mashrooms) не влияет на отображение и используется последовательно, автор предпочитает не блокировать PR; вместо этого предлагать необязательный follow-up issue.
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/ru-RU/_white/cards/danger.ftl:1-1
Timestamp: 2025-10-22T20:31:06.235Z
Learning: В PR по EvacPod для WWhiteDreamProject/wwdpublic: в ru-RU локализациях допустим осознанный сленг сообщества SS13 (напр., «дистры»), если он не ломает отображение и используется последовательно; не блокировать PR, предлагать необязательный follow-up (глоссарий/проверка консистентности).
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: Test Packaging
- GitHub Check: YAML map schema validator
- GitHub Check: YAML Linter
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (28)
Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/medical-responsory.yml (1)
79-79: Изменение соответствует целям PR.Комментирование
SiliconUnshieldedв спискеdamageContainersотключает отображение здоровья для незащищенных КПБ, что согласуется с целью PR по удалению урона от радиации и давления для КПБ.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/security-responsory.yml (1)
79-79: Изменение соответствует целям PR.Комментирование
SiliconUnshieldedсогласуется с целью удаления урона от радиации и давления для КПБ.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/inquisitory.yml (1)
77-77: LGTM!Изменение согласуется с целями PR по удалению урона от радиации и давления для КПБ.
Resources/Prototypes/_NF/Entities/Clothing/Eyes/punk_glasses.yml (1)
76-76: LGTM!Комментирование
SiliconUnshieldedсогласуется с общим паттерном изменений в PR.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/apocryphal.yml (1)
85-85: LGTM!Комментирование
SiliconUnshieldedсогласуется с целями PR. Изменение применено ко всем модкостюмам последовательно.Resources/Prototypes/Damage/containers.yml (1)
38-47: Критическое изменение: удаление контейнера урона SiliconUnshielded подтверждено.Комментирование всего определения
damageContainerс idSiliconUnshieldedкорректно отключает поддержку радиации для КПБ. Различие контейнеров:
Silicon(активный): поддерживает Brute, Heat, ShockSiliconUnshielded(удалён): поддерживал Brute, Heat, Shock, RadiationПроверка подтверждает:
- ✓ Все ссылки на
SiliconUnshieldedзакомментированы или заменены- ✓ Сущности КПБ переключены на контейнер
Silicon(ipc.yml содержитdamageContainer: Silicon)- ✓ Нет активных (незакомментированных) использований
SiliconUnshieldedв кодовой базеResources/Prototypes/Entities/Objects/Tools/welders.yml (1)
125-125: Изменение влияет на механику лечения КПБ — внесено последовательно по всей кодовой базе.Комментирование
SiliconUnshieldedвWeldingHealing.damageContainersозначает, что сварочные инструменты теперь лечат только сущности с контейнером уронаSilicon. КПБ (ИПС) используют контейнер уронаSiliconпосле этих изменений, аSiliconUnshieldedзакомментирован везде в проекте, включая определение контейнера вcontainers.yml.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/leader-responsory.yml (2)
74-79: Изменение выглядит корректно.Отключение
SiliconUnshieldedиз списка контейнеров урона соответствует целям PR по удалению урона от радиации и баротравмы для КПБ. Комментарий с меткой WWDP EDIT сохраняет контекст изменений.
183-183: Отличное исправление форматирования YAML.Исправлена некорректная вложенность списка для
ClothingMaskBreath. Предыдущий формат с двойным дефисом мог вызвать ошибки парсинга YAML.Resources/Prototypes/Entities/Mobs/NPCs/plantbot.yml (1)
47-56: Изменения применены корректно.
SiliconUnshieldedпоследовательно удалён изShowHealthBarsиShowHealthIconsдля plantbot. Изменения соответствуют общей логике PR.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/engineering-responsory.yml (2)
74-79: Изменение корректно.Отключение
SiliconUnshieldedдля шлема инженерного мод-костюма соответствует целям PR.
182-182: Добавление выглядит уместно.Добавлена маска для дыхания в хранилище костюма, что соответствует конфигурации других мод-костюмов.
Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/janitorial-responsory.yml (1)
74-79: Изменение применено корректно.Последовательное отключение
SiliconUnshieldedдля шлема уборочного мод-костюма.Resources/Prototypes/Entities/Objects/Materials/Sheets/glass.yml (1)
53-56: Изменение соответствует целям PR.Удаление
SiliconUnshieldedиз контейнеров урона для лечения КПБ стеклом согласуется с общей логикой отключения данного типа контейнера.Resources/Prototypes/Entities/Clothing/Eyes/hud.yml (2)
35-39: Изменение корректно.Отключение
SiliconUnshieldedдля диагностического HUD согласуется с целями PR.
313-317: Изменение применено последовательно.
SiliconUnshieldedтакже отключён для версии диагностического HUD в виде повязки, что обеспечивает консистентность.Resources/Prototypes/Entities/Objects/Tools/cable_coils.yml (3)
74-78: Изменение выглядит корректно.Отключение
SiliconUnshieldedдля лечения высоковольтными кабелями соответствует общей логике PR.
160-164: Изменение применено последовательно.
SiliconUnshieldedтакже отключён для средневольтных кабелей.
245-249: Изменение применено последовательно.
SiliconUnshieldedотключён для низковольтных кабелей, завершая последовательное обновление всех типов кабелей.Resources/Prototypes/_EE/Entities/NanoTrasen/Modsuits/corporate.yml (1)
78-83: Изменение корректно.Отключение
SiliconUnshieldedдля шлема корпоративного мод-костюма соответствует целям PR.Resources/Prototypes/Entities/Mobs/NPCs/weldbot.yml (1)
47-56: Изменения соответствуют целям PR.Отключение
SiliconUnshieldedиз списковdamageContainersдля компонентов отображения здоровья согласуется с задачей PR по удалению урона от радиации и баротравмы у КПБ. Сварочные боты больше не будут отображать индикаторы здоровья для сущностей с контейнером уронаSiliconUnshielded.Resources/Prototypes/_Goobstation/Entities/Clothing/Eyes/glasses.yml (1)
21-32: LGTM - изменения последовательны.Отключение
SiliconUnshieldedиз компонентовShowHealthBarsиShowHealthIconsдля медико-охранных очков соответствует общей логике PR. Очки сохраняют возможность отображать здоровье биологических, неорганических и кремниевых сущностей.Resources/Prototypes/Traits/physical.yml (4)
743-747: Изменения корректны.Отключение
SiliconUnshieldedв трейтеCyberEyesDiagnosticсоответствует целям PR. Кибернетические диагностические глаза будут отображать здоровье неорганических и кремниевых сущностей, но не сущностей с контейнеромSiliconUnshielded.
784-789: Изменения корректны.Отключение
SiliconUnshieldedв трейтеCyberEyesOmniсогласуется с изменениями в других трейтах и целями PR.
1085-1089: Изменения корректны.Отключение
SiliconUnshieldedв трейтеDiagnosticEyesModuleпоследовательно применяет изменения для модулей глаз КПБ.
1129-1134: Изменения корректны.Отключение
SiliconUnshieldedв трейтеOmniEyesModuleзавершает последовательное применение изменений для всех модулей глаз.Resources/Prototypes/Damage/modifier_sets.yml (1)
399-411: Изменения корректно реализуют удаление радиационной защиты для КПБ.Закомментированные
flatReductionsдля урона от радиации в модификатореIPCнапрямую реализуют цель PR по удалению радиационной защиты у КПБ. Коэффициенты урона остаются без изменений.Resources/Prototypes/Entities/Mobs/Player/ipc.yml (1)
138-143: Удаление баротравмы соответствует целям PR.Закомментированный компонент
Barotraumaполностью удаляет урон от давления для КПБ, что напрямую реализует цель PR. КПБ больше не будут получать тепловой урон от механического перегрева в вакууме.
Resources/Prototypes/_Nuclear14/Entities/Objects/Materials/materials.yml
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/Traits/physical.yml (1)
747-747: Рассмотрите удаление закомментированных строк вместо сохранения их как комментариев.Закомментированные строки
- SiliconUnshieldedсоздают мёртвый код. Поскольку история изменений отслеживается через систему контроля версий, эти строки можно полностью удалить для улучшения читаемости кода.🔎 Предлагаемое исправление
Для строки 747:
damageContainers: - Inorganic - Silicon -# - SiliconUnshielded # WWDP EDITАналогично для строк 789, 1089, 1134.
Also applies to: 789-789, 1089-1089, 1134-1134
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/Traits/physical.yml
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/en-US/_white/cards/hobby.ftl:5-5
Timestamp: 2025-10-22T20:13:31.389Z
Learning: В PR по EvacPod для репозитория WWhiteDreamProject/wwdpublic: если опечатка в ключе локализации (например, card-ep_hobby_mashrooms) не влияет на отображение и используется последовательно, автор предпочитает не блокировать PR; вместо этого предлагать необязательный follow-up issue.
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/ru-RU/_white/cards/danger.ftl:1-1
Timestamp: 2025-10-22T20:31:06.235Z
Learning: В PR по EvacPod для WWhiteDreamProject/wwdpublic: в ru-RU локализациях допустим осознанный сленг сообщества SS13 (напр., «дистры»), если он не ломает отображение и используется последовательно; не блокировать PR, предлагать необязательный follow-up (глоссарий/проверка консистентности).
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
- GitHub Check: YAML Linter
- GitHub Check: YAML map schema validator
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (2)
Resources/Prototypes/Traits/physical.yml (2)
1289-1301: Отключение трейта согласуется с целями PR.Отключение
SuperiorRadiationShieldingлогично, поскольку если КПБ больше не получают урон от радиации (согласно целям PR), трейт защиты от радиации становится избыточным.
526-526: Проверьте, что отключение множественных трейтов соответствует области PR.В дополнение к изменениям, связанным с радиацией и давлением, отключено 9 дополнительных трейтов (
BionicArm,CyberEyesOmni,RestrictedGear,LightAmplification,ThermographicVision,OmniEyesModule,LightAmplificationModule,ThermographicVisionModule,BionicPryArm). Многие из них не имеют прямого отношения к урону от радиации/давления для КПБ.Пожалуйста, подтвердите, что эти отключения являются преднамеренными и входят в область данного PR, либо должны быть вынесены в отдельный PR для изменений баланса.
Also applies to: 753-753, 795-795, 859-859, 877-877, 1095-1095, 1140-1140, 1164-1164, 1258-1258
|
Я не понимаю |
Если я верно понял - оно орет "У вас в игре есть трейт, что уменьшает урон от рады по КПБ! НО!! Вы идиоты убирает им урон от рады вообще!!" |
|
Имхо они должны получать урон от радиации и холода. У них и так есть чистая защита от радиации (которой с перком еще больше), а в космосе они получают урон от холода |
|
@IMaxL02 --> А, ну, я просто отключил трейт. @Remuchi --> Я понимаю, что в SS14 сейчас 2450-2600 год, но не будем забывать, что в космос на данный момент (2025) зачастую отправляют какие-либо аппараты с устройствами, не дающими им замерзать. В каком-нибудь «Вояджере 2» используется РИТГ, не дающий ему замерзать. Почему высокотехнологических КПБ отправляют на станцию? Ради того, чтобы умереть от банальной напасти космоса? Я крайне сомневаюсь. |
|
урон от радиации и космоса ебаный бред как на уровне механа, так и на уровне логики |
|
Я думаю будет лучше полностью откатить ПР с нерфом КПБ |
|
@Spatison, проблема в том, что в том PR'е не только нёрф...
|
RedFoxIV
left a comment
RedFoxIV
left a comment
There was a problem hiding this comment.
Температурный урон в космосе это бред, если ЕЕшники хотят симулировать перегрев - надо выкручивать теплогенерацию, чтобы кпб реально перегревался, а не просто ловил N бёрна в секунду от низкого давления.
Урон от радиации можно оставить. Твой компьютер ирл тебе спасибо не скажет, если ты его ненадолго оставишь рядом с четвёртым реактором.
| # - Heat | ||
| # - Shock | ||
| # - Radiation | ||
| # WWDP EDIT STOP |
| # id: SuperiorRadiationShielding | ||
| # flatReductions: | ||
| # Radiation: 4 | ||
| # WWDP EDIT STOP |
| coefficients: | ||
| Shock: .666 | ||
|
|
| flatReductions: | ||
| Radiation: 4 | ||
| #- type: damageModifierSet | ||
| # id: SuperiorRadiationShielding |
There was a problem hiding this comment.
Тебе кролик указал, что у тебя этот damageModifierSet используется в одном из трейтов.
Ты ничего не поменял, но пометил его доёб, как "резолвед".
Почему?
|
Тут кстати момент в том, что это просто со стороны баланса очень жирно и больно и сильно. Плюс, я даже посмотрел, работал ли компуктер в ваакуме. Оказалось, что нет: "An ordinary PC or laptop won't work at all in vacuum, initially because the hard drive heads need an air cushion to float on. Many people think hard drives are hermetically sealed, but they're not - they just have no through-flow ventilation. In vacuum, a hard drive will instantly eat itself when turned on." Плюс, условный процессор бы просто сгорел нахуй потому что в ваакуме он плохо охлаждается. |
|
А мы разве говорим о простом компьютере, или всё же о высокотехнологичной машинной гуманоидной форме жизни? Люди же не дураки, чтобы работать в космос отправлять тех, кто предназначен только для работы на планете? Но окей, есть ещё вариант с тем, что IPC всё же не NT'шные, а призваны как дешёвая рабочая сила какой-то компании или даже самостоятельные физические лица, которые предназначались изначально вообще не для космоса. Я предлагаю два варианта: в первом, как уже предлагали в обсуждении, делаются трейты за три-пять очков, которые дают изоляцию к космосу и/или радиации, а во втором варианте изоляция уже стоит, но её можно убрать за два-три очка. Изоляция от радиации и космоса как отдельные трейты. От радиации уже есть трейт, осталось лишь сделать от космоса и всё же пересмотреть урон для IPC в космосе, так как просто урон ожогами кажется скудным... |
|
Тут кстати оба правы Решето-корыто, предлагаю решение как с одним из медицинских трейтов:
Один трейт уже есть: Студийный визор. Ему стоит поднять цену до двух, а также добавить "Убирает проблемы с дисплеем в космосе", (Ну и собственно сделать так чтобы в космосе было ебать как неудобно находиться т.к у тебя либо почти ничего не видно, либо цвета колбасит, либо ещё как-то) Это буквально заставит игрунов выбирать между метой и комфортом. Если ты хочешь быть ультра-робаст, ты берёшь себе к примеру наноботы и идёшь ебашиться. Хочешь быть в такси комфорт+? Тогда ты берёшь себе какие-нибудь дебаффы и берёшь все трейты описанные выше, становясь по сути ПОЧТИ человеком, но со своими слабыми и сильными сторонами Машина она прочнее человека, но ей никогда не достичь той же ловкости пальцев, той же скорости обучения и той же чувствительности Кстати да, предлагаю вырезать вообще нахуй все индикаторы хп у КПБ чтобы они рп машина я не знаю когда я сломаюсь, я лишь знаю что я работаю |
7 participants
Описание PR
КПБ теперь не получают урона от давления и радиации!
Медиа
Список
Изменения
🆑 Myaflic