Skip to content
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

Ребаланс ивентов для МРП #2629

Closed
wants to merge 2 commits into from

Conversation

Krosus777
Copy link
Contributor

@Krosus777 Krosus777 commented Sep 26, 2024

Описание PR

  • Ребаланс ивентов для МРП отыгрыша
  • Прототипный фикс бага с "срачем" ивентов. (Так мне и не удалось понять с чем связан баг и почему он только на Корваксе, но ограничение времени частично лечит этот баг, максимум игра спаунит 5 ивентов, а не более 100 как ранее и потом таймеры продолжают корректно отрабатывать)

Почему / Баланс

Текущая частота срабатывания ивентов не подходит для мрп отыгрпыша, каждый раунд превращается в выживание с крупным антагонистом в конце. Данные изменения позволяют увеличить время срабатывания мидраунд ивентов и сократить частоту появления крупных антагонистов на обычных сменах.

Тестировал несколько недель данные изменения на сервере Атара. От игроков получаю только положительные отзывы.

Технические детали

  • Уменьшен шанс появления дракона, время первого спауна перенесено на час двадцать
  • Уменьшен шанс появления одинокого оперативника.
  • Увеличено время между мидраундовыми ивентами
  • Установлено ограничение на выживание в 20 человек(ранее если на сервере менее 5 человек всегда запускалось выживание)
  • Увеличено время срабатывания события метеоритов

Cделано на основании принятой предложки:
https://discord.com/channels/919301044784226385/1278711601201021121/1278711601201021121

Медиа

Требования

Критические изменения

Список изменений

🆑Krosus

  • tweak: Ребаланс мидраудовых ивентов для МРП.
  • fix: Фикс бага со спауном сотни ивентов в секунду.

@Krosus777 Krosus777 requested a review from Morb0 as a code owner September 26, 2024 09:46
@github-actions github-actions bot added Changes: No C# Code has not been changed Status: Needs Review This PR requires new reviews before it can be merged labels Sep 26, 2024
@lzk228
Copy link
Collaborator

lzk228 commented Sep 26, 2024

скилл ишуй

@Vonsant
Copy link
Collaborator

Vonsant commented Sep 26, 2024

скилл ишуй

Ты сам то пытался проверить, почему у нас сервер срет по несколько сотен ивентов в минуту, в то время как у оффов такой проблемы нет?

@kosticia
Copy link
Contributor

kosticia commented Sep 26, 2024

Корвакс ванила, кста

@Vonsant
Copy link
Collaborator

Vonsant commented Sep 26, 2024

Корвакс варила, кста

На ваниле этого бага нет, так что не котируется.

@kosticia
Copy link
Contributor

Корвакс варила, кста

На ваниле этого бага нет, так что не котируется.

На баг пофиг, я про увеличение времени между мидраунд ивентами

@Krosus777
Copy link
Contributor Author

Krosus777 commented Sep 26, 2024

Корвакс варила, кста

На ваниле этого бага нет, так что не котируется.

На баг пофиг, я про увеличение времени между мидраунд ивентами

Ну можешь посмотреть как изменения отрабатывают на Атаре, сейчас загружено в виде прототипа. События срабатываю, но в выживание не превращается. Также если гм видит, что необходимо увеличить интенсивность раунда, то он руками добавляет события, а вот отменить сработанные события он увы уже не может.

@SentryPrimies
Copy link

этим эмэрпэ можно брать и обмазываться

@UmnayaSvinka
Copy link
Contributor

UmnayaSvinka commented Sep 26, 2024

а вот отменить сработанные события он увы уже не может.

удалить антага можно 😕

@Krosus777
Copy link
Contributor Author

Krosus777 commented Sep 26, 2024

а вот отменить сработанные события он увы уже не может.

удалить антага можно 😕

Но вот разрушенную станцию от метеоров и кучу пауков и слаймов - нет.

@lzk228
Copy link
Collaborator

lzk228 commented Oct 13, 2024

какая из частей кода ответственна за фикс?

@lzk228
Copy link
Collaborator

lzk228 commented Oct 16, 2024

как сказал фейнт, обсёр ивентов спрятан в секретах, можешь убрать часть с "фиксом" с этого пра
https://discord.com/channels/919301044784226385/924735650119422032/1295847346999132211

@Morb0
Copy link
Member

Morb0 commented Oct 16, 2024

как сказал фейнт, обсёр ивентов спрятан в секретах, можешь убрать часть с "фиксом" с этого пра https://discord.com/channels/919301044784226385/924735650119422032/1295847346999132211

Реальной связи выявлено не было, это лишь гипотеза

@Krosus777 Krosus777 mentioned this pull request Nov 1, 2024
@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Nov 8, 2024
Copy link
Contributor

github-actions bot commented Nov 8, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label Nov 11, 2024
@Vonsant
Copy link
Collaborator

Vonsant commented Nov 18, 2024

Возможный фикс проходит рассмотрение у визардов.
space-wizards/space-station-14#33222
Если он не поможет, то этот ПР будет принят как временная заплатка, в связи с тем что проблеме уже три месяца.

@cfif126
Copy link
Contributor

cfif126 commented Nov 18, 2024

долой визардов, они там потеряли модсостав

@Vonsant
Copy link
Collaborator

Vonsant commented Nov 18, 2024

долой визардов, они там потеряли модсостав

Я написала кому надо и ПР приняли, ждем апстрима и тестов.

@Krosus777
Copy link
Contributor Author

Возможный фикс проходит рассмотрение у визардов. space-wizards/space-station-14#33222 Если он не поможет, то этот ПР будет принят как временная заплатка, в связи с тем что проблеме уже три месяца.

Как показала практика эти настройки не фиксят проблему, просто шанс что будет спам будет меньше, так как сами ивенты запускаются реже.

@Vonsant
Copy link
Collaborator

Vonsant commented Nov 22, 2024

Проблема более не актуальна.

@Vonsant Vonsant closed this Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Code has not been changed Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants