-
Notifications
You must be signed in to change notification settings - Fork 34
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
[Feature] Stray Supply Pods #171
[Feature] Stray Supply Pods #171
Conversation
WalkthroughИзменения включают добавление новых локализационных записей и прототипов для системы случайных снабженческих капсул в игре. Были обновлены файлы локализации для английского и русского языков, добавлены новые сущности для спавна и управления снабженческими капсулами, а также созданы новые правила игровых событий для появления случайных капсул с грузом. Changes
Suggested labels
Suggested reviewers
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: 3
🧹 Nitpick comments (3)
Resources/Prototypes/_White/Entities/Markers/Spawners/supplypod.yml (1)
34-42
: Проверьте распределение вероятностей редких капсулТекущее распределение:
- 98% шанс на капсулы 5-15 TK
- 2% шанс на капсулы 20-25 TK
Рекомендуется рассмотреть более плавное распределение вероятностей между различными уровнями наград.
- type: RandomSpawner prototypes: - SupplyPodCircleSyndicateSurplus5TK - SupplyPodCircleSyndicateSurplus10TK - SupplyPodCircleSyndicateSurplus15TK - chance: 0.98 + chance: 0.85 rarePrototypes: - SupplyPodCircleSyndicateSurplus20TK - SupplyPodCircleSyndicateSurplus25TK - rareChance: 0.02 + rareChance: 0.15Resources/Locale/ru-RU/_white/prototypes/catalog/supplypod/supplypod.ftl (1)
1-24
: Локализация выглядит правильной!Структура локализации соответствует базовым сущностям и правильно определяет все необходимые строки для новых типов капсул снабжения.
Рекомендуется добавить пустую строку между определениями сущностей для улучшения читаемости кода.
Resources/Prototypes/_White/Catalog/SupplyPod/supply_pod.yml (1)
Line range hint
165-196
: Рекомендуется добавить документациюДля улучшения поддерживаемости кода рекомендуется добавить комментарии, описывающие:
- Логику выбора между обычными и синдикатскими капсулами
- Механику появления капсул
- Влияние на игровой баланс
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
Resources/Locale/en-US/_white/station-events/events/events.ftl
(1 hunks)Resources/Locale/ru-RU/_white/prototypes/catalog/supplypod/supplypod.ftl
(1 hunks)Resources/Locale/ru-RU/_white/prototypes/entities/markers/spawners/supplypod.ftl
(1 hunks)Resources/Locale/ru-RU/_white/station-events/events/events.ftl
(1 hunks)Resources/Prototypes/_White/Catalog/Fills/stray_supply_pod.yml
(1 hunks)Resources/Prototypes/_White/Catalog/SupplyPod/supply_pod.yml
(2 hunks)Resources/Prototypes/_White/Entities/Markers/Spawners/supplypod.yml
(1 hunks)Resources/Prototypes/_White/Entities/Structures/Storage/SupplyPod/base_supply_pod.yml
(1 hunks)Resources/Prototypes/_White/GameRules/events.yml
(1 hunks)
✅ Files skipped from review due to trivial changes (3)
- Resources/Locale/ru-RU/_white/station-events/events/events.ftl
- Resources/Locale/ru-RU/_white/prototypes/entities/markers/spawners/supplypod.ftl
- Resources/Locale/en-US/_white/station-events/events/events.ftl
🔇 Additional comments (5)
Resources/Prototypes/_White/GameRules/events.yml (2)
8-12
: Рекомендуется пересмотреть временные параметры события
Текущие настройки могут привести к слишком частому появлению капсул:
- Раннее время запуска (10 секунд) может быть недостаточным для подготовки игроков
- Короткая задержка повторения (15 секунд) может привести к спаму событий
23-27
: Проверьте баланс синдикатских капсул
Хотя требования к количеству игроков выше (10), время запуска (15 секунд) всё ещё кажется слишком ранним для такого мощного события.
✅ Verification successful
Время запуска синдикатских капсул сбалансировано
Сравнительный анализ параметров событий показывает, что время запуска в 15 секунд для синдикатских капсул является сбалансированным:
- Обычные капсулы снабжения: 10 секунд, 5 игроков минимум
- Синдикатские капсулы: 15 секунд, 10 игроков минимум
- Большинство других опасных событий: 15-25 секунд
Повышенные требования к количеству игроков (10) и увеличенная задержка повторения (40 секунд против 15) компенсируют относительно раннее время запуска.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск прототипов синдикатских предметов для оценки баланса
ast-grep --pattern 'SupplyPodCircleSyndicateSurplus'
Length of output: 52
Script:
#!/bin/bash
# Поиск всех yml файлов с событиями и их содержимого
fd -e yml -e yaml | grep -i event | xargs cat
# Поиск других событий для сравнения параметров
rg "weight:|duration:|minimumPlayers:|earliestStart:|reoccurrenceDelay:" -A 5
Length of output: 71748
Script:
#!/bin/bash
# Поиск других событий для сравнения параметров запуска
rg "weight:|minimumPlayers:|earliestStart:" -B 2 -A 2
Length of output: 67559
Resources/Prototypes/_White/Entities/Structures/Storage/SupplyPod/base_supply_pod.yml (1)
60-61
: Корректное добавление контейнера
Добавление entity_storage
в список контейнеров компонента Construction логично дополняет существующую конфигурацию и соответствует структуре сущности.
Resources/Prototypes/_White/Catalog/SupplyPod/supply_pod.yml (1)
1-29
: Реализация капсулы с грузом выглядит корректной!
Структура сущности правильно наследуется от базовых компонентов и использует таблицу SurplusCargoTable
для заполнения содержимого.
Resources/Prototypes/_White/Catalog/Fills/stray_supply_pod.yml (1)
1-601
: Распределение обычного содержимого выглядит сбалансированным
Структура таблицы содержимого хорошо организована с правильным распределением весов для различных категорий предметов. Все базовые категории снабжения представлены.
Описание PR
Сбившиеся со своего пути грузовые капсулы падают на станцию, могут содержать от всего ассортимента карго, до полноценных наборов синдиката ценой от 5ТК до 25ТК.
Медиа
Список
Изменения
🆑 PuroSlavKing