-
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
Fix psi v3 #1074
Fix psi v3 #1074
Conversation
WalkthroughИзменения затрагивают несколько компонентов системы Psionics. В методах обновления и запуска событий добавлены проверки на состояние паузы сущностей, что позволяет пропускать их обработку. Также изменены количественные параметры реагентов в YAML файлах, обновлены свойства событий и добавлен новый реагент в руководство Psionics. Changes
Sequence Diagram(s)sequenceDiagram
participant System as GlimmerStructuresSystem
participant Entity as GlimmerSourceEntity
participant Meta as MetaDataComponent
System->>Entity: Получить сущность с MetaData
Entity-->>System: Передача данных сущности
System->>System: Проверить Paused(owner, MetaData)
alt Сущность на паузе
System->>System: Пропустить обработку
else Сущность активна
System->>Entity: Обработка накопления glimmer
end
sequenceDiagram
participant Rule as SpawnRule (Revenant/Wisp)
participant Src as Source Entity
Rule->>Src: Итерация по сущностям
Src-->>Rule: Передача данных
Rule->>Rule: Проверка Paused(source)
alt Сущность на паузе
Rule->>Rule: Пропустить дальнейшую обработку
else Сущность активна
Rule->>Src: Выполнить логику спауна
end
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (4)
⏰ Context from checks skipped due to timeout of 300000ms (3)
🔇 Additional comments (8)
✨ Finishing Touches
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: CHILL
Plan: Pro
📒 Files selected for processing (2)
Content.Shared/Backmen/Chapel/Components/SacrificialAltarComponent.cs
(2 hunks)Resources/Prototypes/_Backmen/GameRules/events.yml
(6 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- Resources/Prototypes/_Backmen/GameRules/events.yml
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (3)
Content.Shared/Backmen/Chapel/Components/SacrificialAltarComponent.cs (3)
2-2
: Улучшение типизации для повышения безопасности типовЗамена строковых идентификаторов на строго типизированные
ProtoId
иEntProtoId
повышает надежность кода и позволяет обнаруживать ошибки на этапе компиляции.Also applies to: 6-6, 25-25, 46-46
28-28
: Значительное увеличение шансов получения наградыШансы получения награды были существенно увеличены:
- Базовый шанс: с 10% до 30%
- Шанс для пользователей Библии: с 50% до 80%
Рекомендуется проверить баланс игры с новыми значениями.
Also applies to: 31-31
34-34
: Увеличение диапазона наград блюспейсДиапазон наград был увеличен с 1-4 до 4-8, что может существенно повлиять на игровой баланс.
Предлагаю проверить, не приведет ли это к избытку ресурсов в игре.
Also applies to: 37-37
|
||
[DataField("glimmerReductionMin")] | ||
public int GlimmerReductionMin = 30; | ||
public int GlimmerReductionMin = 500; |
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.
Существенное увеличение значений уменьшения глиммера
Значения уменьшения глиммера были значительно увеличены:
- Минимум: с 30 до 500
- Максимум: с 60 до 900
Это радикальное изменение может сильно повлиять на механику глиммера.
Рекомендуется:
- Проверить влияние на общий баланс системы глиммера
- Убедиться, что такое большое уменьшение не нарушает игровую механику
- Рассмотреть возможность постепенного увеличения значений
Also applies to: 43-43
🆑
Summary by CodeRabbit
Refactor
New Features
Documentation