[Balance] Race of races #1020
[Balance] Race of races #1020
Conversation
Обзор измененийWalkthroughВ этом обновлении были внесены изменения в несколько прототипов сущностей и наборов модификаторов урона. Добавлены новые компоненты для сущностей дварфов, комбайнов и фелинидов, включая модификаторы скорости движения и обработки урона. Также созданы три новых набора модификаторов урона: CombineHuman, FelinidSkin и DwarfSkin с различными коэффициентами для разных типов повреждений. Changes
Poem
Tip 🌐 Web search-backed reviews and chat
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 300000ms (3)
🔇 Additional comments (1)
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 (
|
![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
🧹 Nitpick comments (2)
Resources/Prototypes/Entities/Mobs/Species/dwarf.yml (1)
21-24: Рекомендуется скорректировать пороги замедленияТекущие пороги замедления могут быть недостаточными для баланса. Предлагается добавить дополнительный порог для более плавного снижения скорости.
- type: SlowOnDamage speedModifierThresholds: + 100: 0.5Resources/Prototypes/_Backmen/Damage/modifier_sets.yml (1)
131-162: Необходима общая корректировка модификаторов уронаТекущие наборы модификаторов создают слишком резкие различия между расами:
- CombineHuman: Полные иммунитеты нереалистичны
- FelinidSkin: Множители урона слишком высоки
- DwarfSkin: Требуется дифференциация типов урона
Рекомендации по балансировке:
- Заменить нулевые значения на 0.2-0.4 для CombineHuman
- Уменьшить множители FelinidSkin до 1.15-1.25
- Дифференцировать значения DwarfSkin (например, больше устойчивости к Blunt, меньше к Piercing)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
Resources/Prototypes/Entities/Mobs/Species/dwarf.yml(1 hunks)Resources/Prototypes/_Backmen/Damage/modifier_sets.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Mobs/Player/combine.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Mobs/Species/felinid.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
| @@ -34,6 +34,7 @@ | |||
| prototype: Felinid | |||
| - type: Damageable | |||
| damageContainer: Biological | |||
| damageModifierSet: FelinidSkin | |||
There was a problem hiding this comment.
🛠️ Refactor suggestion
Необходимо смягчить уязвимости фелинидов
Текущие множители урона (1.35-1.45) делают фелинидов чрезмерно уязвимыми. Учитывая их способности к скрытности, рекомендуется уменьшить множители до 1.15-1.25.
- type: Damageable
- damageModifierSet: FelinidSkin
+ damageModifierSet: FelinidSkinBalanced # Требуется создать новый набор модификаторовCommittable suggestion skipped: line range outside the PR's diff.
| - type: MovementSpeedModifier #backmen: Race of race | ||
| baseSprintSpeed: 4 |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Скорость дварфов требует корректировки
Базовая скорость спринта 4 в сочетании с высокой устойчивостью к урону может сделать дварфов слишком эффективными. Рекомендуется уменьшить до 3.5.
- type: MovementSpeedModifier
- baseSprintSpeed: 4
+ baseSprintSpeed: 3.5📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - type: MovementSpeedModifier #backmen: Race of race | |
| baseSprintSpeed: 4 | |
| - type: MovementSpeedModifier #backmen: Race of race | |
| baseSprintSpeed: 3.5 |
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Damage/modifier_sets.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
| - type: damageModifierSet | ||
| id: FelinidSkin | ||
| coefficients: | ||
| Blunt: 1.45 | ||
| Slash: 1.35 | ||
| Piercing: 1.35 | ||
| Heat: 1.2 | ||
| Cold: 1.5 | ||
| Cellular: 1.1 | ||
| Bloodloss: 1.25 | ||
| Shock: 1.25 | ||
| Radiation: 1.1 | ||
|
|
There was a problem hiding this comment.
Требуется балансировка FelinidSkin.
Текущие модификаторы урона делают расу слишком уязвимой:
- Коэффициент 1.45 для тупого урона слишком высок
- Все коэффициенты выше 1.0, что делает расу уязвимой ко всем типам урона
- Отсутствуют компенсирующие преимущества
Рекомендуемые изменения:
coefficients:
- Blunt: 1.45
- Slash: 1.35
- Piercing: 1.35
+ Blunt: 1.2
+ Slash: 1.1
+ Piercing: 1.1
Heat: 1.2
Cold: 1.5
- Cellular: 1.1
- Bloodloss: 1.25
- Shock: 1.25
- Radiation: 1.1
+ Cellular: 0.9
+ Bloodloss: 1.1
+ Shock: 1.1
+ Radiation: 0.9📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - type: damageModifierSet | |
| id: FelinidSkin | |
| coefficients: | |
| Blunt: 1.45 | |
| Slash: 1.35 | |
| Piercing: 1.35 | |
| Heat: 1.2 | |
| Cold: 1.5 | |
| Cellular: 1.1 | |
| Bloodloss: 1.25 | |
| Shock: 1.25 | |
| Radiation: 1.1 | |
| - type: damageModifierSet | |
| id: FelinidSkin | |
| coefficients: | |
| Blunt: 1.2 | |
| Slash: 1.1 | |
| Piercing: 1.1 | |
| Heat: 1.2 | |
| Cold: 1.5 | |
| Cellular: 0.9 | |
| Bloodloss: 1.1 | |
| Shock: 1.1 | |
| Radiation: 0.9 | |
| - type: damageModifierSet | ||
| id: CombineHuman | ||
| coefficients: | ||
| Blunt: 0.8 | ||
| Slash: 0.8 | ||
| Piercing: 0.6 | ||
| Stun: 0.5 | ||
| Cold: 0.0 | ||
| Poison: 0.0 | ||
| Radiation: 0.0 | ||
| Asphyxiation: -1.0 | ||
| Bloodloss: 0.0 | ||
| Cellular: 0.0 | ||
|
|
There was a problem hiding this comment.
Необходимо пересмотреть баланс модификаторов CombineHuman.
Текущая реализация создает потенциально имбалансный набор модификаторов:
- Отрицательный коэффициент для Asphyxiation (-1.0) означает лечение от урона удушьем
- Полный иммунитет к нескольким типам урона (Cold, Poison, Radiation, Bloodloss, Cellular)
Предлагаемые изменения:
coefficients:
Blunt: 0.8
Slash: 0.8
Piercing: 0.6
Stun: 0.5
- Cold: 0.0
- Poison: 0.0
- Radiation: 0.0
- Asphyxiation: -1.0
- Bloodloss: 0.0
- Cellular: 0.0
+ Cold: 0.3
+ Poison: 0.3
+ Radiation: 0.3
+ Asphyxiation: 0.5
+ Bloodloss: 0.3
+ Cellular: 0.3📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - type: damageModifierSet | |
| id: CombineHuman | |
| coefficients: | |
| Blunt: 0.8 | |
| Slash: 0.8 | |
| Piercing: 0.6 | |
| Stun: 0.5 | |
| Cold: 0.0 | |
| Poison: 0.0 | |
| Radiation: 0.0 | |
| Asphyxiation: -1.0 | |
| Bloodloss: 0.0 | |
| Cellular: 0.0 | |
| - type: damageModifierSet | |
| id: CombineHuman | |
| coefficients: | |
| Blunt: 0.8 | |
| Slash: 0.8 | |
| Piercing: 0.6 | |
| Stun: 0.5 | |
| Cold: 0.3 | |
| Poison: 0.3 | |
| Radiation: 0.3 | |
| Asphyxiation: 0.5 | |
| Bloodloss: 0.3 | |
| Cellular: 0.3 |
|
нашёлся любитель дварфов |
а не слишком ли имбово будет? фелиниды не должны быть боевой расой и у них буквально интегрированные перчатки вора + могут прятаться в сумках |
Пусть будут стелс рассой, могут быстро красться и воровать, но при этом слабы И вообще я думаю вместо резистов лучше тогда ничего не менять и просто уменьшить порог крита до 90 |
|
Предлагаю сделать феленидав тупой урон 2.0, а дворфам весь урон 0.5, а людям член 30 см, радм баланса вселенной. |
Описание PR
меняет баланс рас
Тип PR
Summary by CodeRabbit
Новые функции
Damageableдля сущности "Урист Макбайн" с модификатором урона "Комбайн"MobThresholdsдля сущности "Урист МакФелинид" с порогами состоянияУлучшения