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

Map weight #11849

Merged
merged 6 commits into from Aug 10, 2023
Merged

Map weight #11849

merged 6 commits into from Aug 10, 2023

Conversation

KandreyKandrey
Copy link
Contributor

@KandreyKandrey KandreyKandrey commented Aug 5, 2023

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

После начала голосования за след. карту берём 3 последних раундов (НЕ включая раунд, в котором проводится голосование) и за каждый раунд на одной карте забираем у неё 20% веса.
Box и Snowy Box считаем одной картой (одинаково понижаем им вес), но не ниже 0.4.
т.е если Box был 3 раунда подряд, то у снежного бокса вес будет тоже 0.4, а не 0.7(его стандартный вес) * 0.4 = 0.28

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

close #11844 (наверное)

Авторство

Идея частично обсуждалась в #11438

Чеинжлог

🆑 Kandrey

  • experiment: Чем чаще выбирается карта, тем меньше будет её вес на голосовании.

@TauKitty
Copy link
Contributor

TauKitty commented Aug 5, 2023

Changelog status: ✔️

@TauKitty
Copy link
Contributor

TauKitty commented Aug 7, 2023

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/boxstation/40389/49

@volas
Copy link
Member

volas commented Aug 8, 2023

Единственное что, возможно потребуется пересмотр дефолтных весов из конфига.

  • фалкон имеет 1.5, на фоне понижения веса других карт за повтор будет еще более ультимативно
  • зимний бокс имеет по дефолту и не по праздникам 0.7, если хотя бы половина из 10 последних раундов была на мейнтрим боксе - зимний бокс будет сложно выбрать. Может быть имеет смысл сделать минимальный порог.

@AgRevol
Copy link
Contributor

AgRevol commented Aug 8, 2023

Если честно я бы возможно пересмотрел 10 раундов. 10 раундов это целый день игры, люди придут вечером и не смогут поиграть любимую карту из-за того что когда онлайн был 20 человек - там стоял условный бокс. Я бы может пересмотрел систему на 2-3 последних раунда но больший коэфециент.
Ну и минимальный порог как предложил Волас тоже считаю хорошей идеей

@volas
Copy link
Member

volas commented Aug 8, 2023

про целый день тоже хорошая мысль, если уменьшать страйк - я бы тогда еще добавил в запрос проверку на end_state чтоб считать только настоящие раунды. Валидные значения тогда (там хардкод, да...): proper completion, nuke

@Deahaka
Copy link
Contributor

Deahaka commented Aug 8, 2023

★ Строечка для отсутствия онлайна
★★ Фалькон для маленького онлайна
★★★ Бокс для обычного
★★★ Снежный Бокс для обычного
★★★ Прометей для обычного
★★★★ Гамма для большого онлайна
★★★★ Дельта для большого онлайна

При этом Бокс и Снежный Бокс считаются одной группой для понижения коефициента. Дельта и Гамма не конкурируют с Боксом/Айсбоксом и Прометеем по онлайну.
То есть выбор у игроков сейчас между: Боксом, Прометеем и штрафным Снежным Боксом, а будет... точно также только у Прометея коефициент выше, а у Снежного Бокса ещё меньше.

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

@KandreyKandrey
Copy link
Contributor Author

Дельта и Гамма не конкурируют с Боксом/Айсбоксом и Прометеем по онлайну.

Бокс/Прометей сделаны для меньшего онлайна, но при большом не выпадают из воута

@AgRevol
Copy link
Contributor

AgRevol commented Aug 8, 2023

Дельта и Гамма не конкурируют с Боксом/Айсбоксом и Прометеем по онлайну.

Бокс/Прометей сделаны для меньшего онлайна, но при большом не выпадают из воута

Разделение на большой/средний онлайн я лично считаю немного лишними при голосовании. Если игроки хотят играть на пустой станции - их право, онлайн может и позже зайти. Главное не пихать людей на карты где им будет тесно.
Ну и объективно гамма будет комфортно играться и с 60 онлайна, как и дельта, скорее всего

@Deahaka
Copy link
Contributor

Deahaka commented Aug 8, 2023

Если смотреть на Фалькон, то множитель на его пик (1.5 вроде?) выглядит вкуснее, чем без него. А всего лишь достаточно поставить кап на плееров минимальный (например 60 те же). Если Гамма будет в воуте на постоянке, я не против сделать только Дельте ~1.5-2 множитель при кол-ве игроков > 60.

@KandreyKandrey
Copy link
Contributor Author

поменял описание

основные изменения:

  • берём 3 последних раунда (раунд, в котором проводится голосование не берём)
  • берём только раунды, завершившиеся в состоянии 'proper completion' или 'nuke'
  • раунд, в котором проводится голосование не берём, т.к на тот момент он ещё не завершился -> в end_state будет null
  • за каждый раунд на карте забираем 20% от стандартного веса (т.е у фалкона 20% от его 2, у снежного бокса 20% от его 0.7)
  • минимальный вес ставим 0.4 (т.е вес снежного бокса не опустится ниже)

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Aug 9, 2023
@volas volas merged commit 619fe8c into TauCetiStation:master Aug 10, 2023
10 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Aug 10, 2023
vc.text = VM.GetFullMapName()
if(VM.voteweight != 1)
vc.text += "\[vote weight: [VM.voteweight]\]"
vc.mapname = VM.map_name
vc.vote_weight = VM.voteweight
choices.Add(vc)


/datum/poll/nextmap/proc/get_voteweights()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/boxstation/40389/61

@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/golosovanie-golosovaniya-za-kartu/41114/1

@Deahaka Deahaka mentioned this pull request Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Experimental Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Proposal] Удалить Box station из ротации на неопределенное время
5 participants