-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Маленькие изменения #309
Маленькие изменения #309
Conversation
WalkthroughВ данном запросе на изменение внесены модификации в файлы конфигурации YAML для различных сущностей. В файле Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CableStack
participant HolosignSecurity
User->>CableStack: Запрос на использование
CableStack->>User: Урон Heat: -3.50, Shock: -3.50
User->>HolosignSecurity: Запрос на взаимодействие
HolosignSecurity->>User: Не уничтожается автоматически
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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
- Resources/Prototypes/Entities/Objects/Tools/cable_coils.yml (1 hunks)
- Resources/Prototypes/Entities/Structures/Holographic/projections.yml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Prototypes/Entities/Objects/Tools/cable_coils.yml (1)
Pattern
**/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:
, можно оставить оригинальную часть с- id:
.Resources/Prototypes/Entities/Structures/Holographic/projections.yml (1)
Pattern
**/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:
, можно оставить оригинальную часть с- id:
.
🪛 yamllint
Resources/Prototypes/Entities/Objects/Tools/cable_coils.yml
[error] 38-38: wrong indentation: expected 6 but found 4
(indentation)
[warning] 40-40: too few spaces before comment
(comments)
🔇 Additional comments (2)
Resources/Prototypes/Entities/Structures/Holographic/projections.yml (1)
71-72
: 🧹 Nitpick (assertive)Проверьте влияние на игровой баланс.
Отключение компонента
TimedDespawn
сделает голографический барьер постоянным, что может повлиять на игровой баланс. Убедитесь, что:
- Игроки имеют достаточно способов противодействия постоянным барьерам
- Количество размещаемых барьеров ограничено другими механиками
✅ Verification successful
Изменение баланса приемлемо
Удаление компонента
TimedDespawn
компенсируется следующими механиками:
- Голопроектор требует энергоячейку для работы
- Каждое использование потребляет 120 единиц заряда
- Барьеры описаны как "хрупкие" и могут быть уничтожены
- Количество барьеров ограничено зарядом батареи
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверяем наличие других механик ограничения барьеров rg -l "HolosignSecurity" | xargs rg "maxAmount|maxCharges|cooldown"Length of output: 67
Script:
#!/bin/bash # Search for holosign security barrier related files and configurations rg -l "HolosignSecurity|holosign|holobarrier" --type yaml # Then let's look at the content of these files fd -e yml -e yaml . | xargs rg "HolosignSecurity|holosign|holobarrier" -C 5Length of output: 2714
Script:
#!/bin/bash # Let's check the parent entity and base holoprojector configuration rg "parent: Holoprojector" -B 10 -A 10 Resources/Prototypes/Entities/Objects/Devices/holoprojectors.yml # Also check for any power cell or charge limitations rg "chargeUse|maxCharges|PowerCell" Resources/Prototypes/Entities/Objects/Devices/holoprojectors.yml -C 3Length of output: 4219
Resources/Prototypes/Entities/Objects/Tools/cable_coils.yml (1)
36-42
: Проверьте значительное увеличение силы леченияИзменения увеличивают силу лечения более чем в два раза (с -1.66 до -3.50). Это существенное изменение баланса, которое может иметь непредвиденные последствия.
Пожалуйста, подтвердите:
- Было ли это изменение протестировано на предмет баланса?
- Не делает ли это кабели слишком эффективным средством лечения?
✅ Verification successful
Увеличение силы лечения обосновано для кабелей
Изменение значений лечения с -1.66 до -3.50 для типов урона Heat и Shock является обоснованным, так как:
- Кабели - единственный предмет, способный лечить повреждения контейнера Silicon
- Это специализированный инструмент для ремонта синтетических существ (киборгов, ИПК)
- Значения соответствуют общему балансу, учитывая:
- Ограниченное применение (только для Silicon)
- Задержку использования 0.7 секунд
- Необходимость иметь запас кабелей
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for other healing components to compare values rg -A 5 "Healing:\n.*damage:\n.*types:" --type yamlLength of output: 245
Script:
#!/bin/bash # Search for damage/healing configurations in yaml files rg -U --multiline "damage:\n.*types:.*" --type yaml -A 3Length of output: 307361
Script:
#!/bin/bash # Search specifically for healing-related components and effects rg -U --multiline "components:\n.*Healing:" --type yaml -A 10Length of output: 63
Script:
#!/bin/bash # Look for other cable-related configurations fd "cable.*\.yml$" | xargs catLength of output: 11708
Script:
#!/bin/bash # Search for healing-related components and effects with a different pattern rg -U "Healing:.*damage:" --type yaml -A 5Length of output: 44
Script:
#!/bin/bash # Search for other healing items/effects to compare values rg -U "damage:.*types:.*Heat:.*-[0-9]" --type yamlLength of output: 52
Script:
#!/bin/bash # Look for other silicon-related healing rg -U "damageContainers:.*Silicon" --type yamlLength of output: 48
Script:
#!/bin/bash # Search for healing effects with different patterns rg -U "types:.*Heat:.*-[0-9]|types:.*Shock:.*-[0-9]" --type yaml -A 3Length of output: 71
Script:
#!/bin/bash # Look for silicon-related components with a different pattern rg -U "Silicon.*damage|damage.*Silicon" --type yaml -A 5Length of output: 1674
Script:
#!/bin/bash # Search for healing components with broader pattern rg -U "components:.*Healing|Healing:.*damage" --type yaml -A 10Length of output: 65
🧰 Tools
🪛 yamllint
[error] 38-38: wrong indentation: expected 6 but found 4
(indentation)
[warning] 40-40: too few spaces before comment
(comments)
@DarkSide-droids верни деспавн голопроекту, но повысь его настолько, насколько считаешь нужным, не более 10 минут |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- Resources/Prototypes/Entities/Structures/Holographic/projections.yml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Prototypes/Entities/Structures/Holographic/projections.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
🪛 yamllint
Resources/Prototypes/Entities/Structures/Holographic/projections.yml
[warning] 72-72: too few spaces before comment
(comments)
Описание обновления:
Голобарьер службы безопасности, одна из тех вещей которая так редко используется в игре. Почему же? Скорее всего, из-за того что он слишком быстро пропадает. 180 секунд - крайне мало для такой вещи. Может быть после этого PR-а, он станет более полезным.
Провода для ремонта КПБ получили небольшое улучшение. Ранее чтобы починить 120 повреждений на КПБ, уходило почти 60-70 проводов. Теперь же, потребуется немного меньше проводов для ремонта
Медиа
Увы.... (Нет)
Проверки
Изменения
🆑 MrDarkSide