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

Mode declares #12954

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Mode declares #12954

wants to merge 8 commits into from

Conversation

Chip11-n
Copy link
Contributor

@Chip11-n Chip11-n commented Mar 14, 2024

Я тут в полуживом, полуобморочном состоянии, потому пока что буду выкатывать ПРы из запаса. ПР Киборга, возможно, исправил бы ситацию получше, но как есть.

Описание изменений

Демократично выбирают тимбазу, выпадает экста за якобы неимением режимов способных запуститься

if(!runnable_modes.len)
			runnable_modes = config.get_always_runnable_modes()

ПР изменяет, что в случае не нахождения режима, экста не ставится, а возвращает в лобби, запускает воут на режим, и пробуем ещё раз. Если реди вообще нет, только тогда запустится экста.

В дополнение:

  • ООС не выключается, если раунд не сумел запуститься.
  • Немедленный старт слетает если есть реди, но раунд не сумел запуститься, предотвращая спам и ввиду сложности отключения оного при таком.
  • Экста может запуститься несколько раз подряд. Да, раньше это происходило из-за костыля.

Почему и что этот ПР улучшит

closes #12865

Авторство

Чеинжлог

🆑

  • bugfix: Экста не может более запуститься, если её не выбрали.
  • bugfix: ООС не выключается, если раунд не запустился.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@KIBORG04
Copy link
Member

Думаю в ченджлоге стоит пояснить, что теперь смотрятся не реди, а кол-во игроков.

Copy link

github-actions bot commented Apr 4, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@volas volas added Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже and removed Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже labels Apr 8, 2024
@volas
Copy link
Member

volas commented Apr 8, 2024

я сейчас не хочу погружаться в эту часть кода, а апрув так никто и не поставил. Стоит в ТМ вешать?

@volas
Copy link
Member

volas commented Apr 8, 2024

balance: Режимы при запуске смотрят на общее количество игроков, а не их готовность.

режимы скалируют свои роли от количества игроков, это может потенциально что-то сломать. Типо старт раунда с режимом, который потом зафейлится, потому что вдруг нет игроков. Но я код не смотрел.

Выпадающая экста тоже не должно быть чем-то плохим имхо, тем более что эксты у нас давно не эксты и рандом может кучу всего в раунде еще наспавнить

@Chip11-n
Copy link
Contributor Author

Chip11-n commented May 3, 2024

Выпадающая экста тоже не должно быть чем-то плохим имхо, тем более что эксты у нас давно не эксты и рандом может кучу всего в раунде еще наспавнить

Хоть сколько-то серьезные антаги выпадают только в середине раунда, да и как-то не очень: выбираешь командные роли, а раунд по итогу экста. Даже в секрете такого не происходит.

Copy link

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@KIBORG04
Copy link
Member

ты же жив? Можно ТМить?

@Chip11-n
Copy link
Contributor Author

Я пока только-только начинаю вновь просыпаться. А в ТМ точно можно.

@KIBORG04 KIBORG04 added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label May 26, 2024
@volas volas removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jun 2, 2024
@Udokun
Copy link
Contributor

Udokun commented Jun 2, 2024

#63887

@volas
Copy link
Member

volas commented Jun 2, 2024

^ раунд выше выпал в цикл невозможности выбора режима

@volas
Copy link
Member

volas commented Jun 2, 2024

впрочем, после отката пр-а и рестарта оно тоже затупило. Почему-то в пулле секрета был только тритор, и на него не хватало игроков. Может из-за last gamemode оттуда просто вылетел экстендет, не знаю.

@Chip11-n
Copy link
Contributor Author

Chip11-n commented Jun 15, 2024

Это бы повторно в ТМ кинуть

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Config Update Fix Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Экста в тимбазе
6 participants