-
Notifications
You must be signed in to change notification settings - Fork 154
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
[Balance] спидозники #1004
[Balance] спидозники #1004
Conversation
ОбзорWalkthroughЭтот pull request вносит изменения в локализацию и прототипы болезней. В файле локализации добавлен новый идентификатор болезни Changes
Poem
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
🧹 Nitpick comments (2)
Resources/Prototypes/_Backmen/Diseases/infectious.yml (2)
424-429
: Улучшите механику лечения постельным режимом!Текущая реализация
DiseaseBedrestCure
дляGreenFlu
работает только на начальных стадиях (0-1). Рекомендуется расширить эффективность постельного режима на все стадии болезни для предоставления альтернативного метода лечения.stages: - 0 - 1 + - 2 + - 3
373-389
: Добавьте уникальные симптомы!Первая стадия
GreenFlu
копирует симптомы обычной простуды. Рекомендуется добавить уникальные симптомы для лучшей идентификации болезни.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Resources/Locale/ru-RU/backmen/disease/disease-proto.ftl
(1 hunks)Resources/Prototypes/_Backmen/Diseases/infectious.yml
(8 hunks)
🔇 Additional comments (3)
Resources/Locale/ru-RU/backmen/disease/disease-proto.ftl (2)
19-19
: Локализация соответствует конвенции именования!Новая локализация "зеленый грипп" соответствует общему стилю именования болезней в файле.
22-22
: Проверьте необходимость дублирования названия!Обратите внимание, что "Ксеновирус" уже присутствует в строке
disease-proto-xeno
. Возможно, следует использовать другое название дляdisease-proto-xenodrone
, чтобы избежать путаницы.Resources/Prototypes/_Backmen/Diseases/infectious.yml (1)
363-437
: Проверьте баланс новой болезни!Болезнь
GreenFlu
имеет несколько потенциальных проблем:
- Слишком высокий урон на последней стадии (5 ед. урона с вероятностью 0.3)
- Высокое сопротивление лечению (cureResist: 0.5)
- Быстрая прогрессия между стадиями
Рекомендации:
- Уменьшить урон на последней стадии
- Снизить сопротивление лечению
- Увеличить время между стадиями
- cureResist: 0.5 + cureResist: 0.3 - probability: 0.3 + probability: 0.2 damage: types: - Poison: 5 + Poison: 3
- !type:DiseaseReagentCure | ||
reagent: | ||
ReagentId: Ultravasculine |
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.
🛠️ Refactor suggestion
Проверьте баланс универсального лекарства!
Ultravasculine
теперь лечит слишком много болезней. Это может сделать игровой процесс менее интересным, так как игрокам достаточно будет иметь только одно лекарство для лечения большинства заболеваний.
Рекомендуется:
- Разделить лечение между разными препаратами
- Добавить побочные эффекты для
Ultravasculine
- Сделать лечение более сложным, требующим комбинации препаратов
Also applies to: 96-98, 129-129, 217-227, 299-301, 338-338, 359-361
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.
Ну вроде сойдёт
Описание PR
Добавляет новые возможности выздоровления и новый вирус
Тип PR
Изменения
🆑 no_mad
Summary by CodeRabbit
Новые функции
Исправления