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

Убирает штрафы и бонусы от скиллов на лоупопе #13644

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

Conversation

simb11
Copy link
Contributor

@simb11 simb11 commented Nov 24, 2024

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

Если количество реди на начало раунда меньше 11, раунд начнётся с отключённой системой умений.
т.е. ассистент сможет оперировать с такой же скоростью как и медик, чинить стены с такой же скоростью как и инженер и ТД

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

Игру на сервере с небольшим онлайном.

Авторство

Чеинжлог

🆑 Simbaka

  • tweak: Отключение системы штрафов и бонусов от скиллов в раундах с малым онлайном.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added the Tweak label Nov 24, 2024
@TauKitty
Copy link
Contributor

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

https://forum.taucetistation.org/t/obsuzhdenie-pra-tihookeanskij-rubezh-skilov/44115/8

@DarthSidiousPalpatine
Copy link
Contributor

Вот это збс. Богоугодно. Сначала потыкать в лоупоп скиллы, а потом мб поймём что скиллы в принципе кринж особенно когда админам пох и они всё равно правило меты юзают.

@DarthSidiousPalpatine
Copy link
Contributor

Ток исполнение какое-то странное, имхо.

@simb11
Copy link
Contributor Author

simb11 commented Nov 24, 2024

Ток исполнение какое-то странное, имхо.

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

а так, играться будет как раньше (до ввода скиллов)

@DarthSidiousPalpatine
Copy link
Contributor

DarthSidiousPalpatine commented Nov 24, 2024

Ток исполнение какое-то странное, имхо.

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

а так, играться будет как раньше (до ввода скиллов)

Я бы для начала сделал на фалконе свои профы совмещённые со скиллами нормальными.

Ну и вообще... 20 человек играют, 9 прожимают реди - игра говорит лоупоп.

@simb11
Copy link
Contributor Author

simb11 commented Nov 24, 2024

Я бы для начала сделал на фалконе свои профы совмещённые со скиллами нормальными.

так там уже совмещённые профы, например медики замещают сразу весь медбей и имеют хорошие навыки во всех областях (даже не связанных с медициной)

Ну и вообще... 20 человек играют, 9 прожимают реди - игра говорит лоупоп.

ну вот вчера было 25 человек в лобби, а нажало реди всего 4, потом подключилось пара человек и все остальные ливнули.

@DarthSidiousPalpatine
Copy link
Contributor

DarthSidiousPalpatine commented Nov 24, 2024

так там уже совмещённые профы, например медики замещают сразу весь медбей и имеют хорошие навыки во всех областях (даже не связанных с медициной)

А надо больше совмещения. Например - медики + ученые. Каргонцы + Инженеры. Ну и сервис профы все в одну сделать. Обслугой назвать. И они могут и в баре сидеть, и выращивать, и готовить, и убирать, и в церкви отмаливать грехи.

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

@@ -21,7 +23,7 @@
return do_after(user, delay = apply_skill_bonus(user, delay, required_skills, multiplier), target = target, extra_checks = extra_checks)

/proc/handle_fumbling(mob/user, atom/target, delay, required_skills, message_self = "", text_target = null, check_busy = TRUE, can_move = FALSE)
if(is_skill_competent(user, required_skills))
if(is_skill_competent(user, required_skills) || SSticker.is_lowpop)
Copy link
Member

Choose a reason for hiding this comment

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

быстрые проверки в таких местах лучше делать первыми, не нужно сверять листы скиллов если мы в лоупоп и и так входим в условие

@volas
Copy link
Member

volas commented Nov 24, 2024

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

Тогда тут это был бы аспект со 100% шансом выпадения для лоупоп раунда, и игрокам бы это тоже понятно писалось, что так и так, в раунде действует аспект на безскиллов.

Не реквест, просто мысли.

@ZER0-In
Copy link

ZER0-In commented Nov 24, 2024

Ток исполнение какое-то странное, имхо.

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

а так, играться будет как раньше (до ввода скиллов)

Мне нравятся скиллы. Когда нормальный или хайпоп онлайн.
Но для лоупопа отрубать скиллы - я об это трубил еще когда только в тм закинули систему навыков. Я кричал о том, что второй сервер умрет, так как там пару человек без скиллов будут люто душиться лишь чтобы запустить станцию на нормальную работу.

Давно надо было ввести эту систему или как я предлагал - отрубить скиллы для второго сервера.

Насчет этого ПРа, он должен был выйти сразу вместе со скиллами хуй знает когда назад. Насчет 11 человек? Не мало? Может порог 20 сделать. Как это работает, если зайдет человек больше лимита? Раунд старт на 11, а потом заходят еще 5. Для новых людей скиллы работают или нет? Или система снова включается для всех. (Со стороны кода я предполагаю, что это зафиксировано за раундом, что если начали 11, а потом зашло еще 50 - раунд все равно будет у всех без ограничений на скиллы).

@simb11
Copy link
Contributor Author

simb11 commented Nov 24, 2024

если начали 10, а потом зашло еще 50 - раунд все равно будет у всех без ограничений на скиллы

да

@Udokun
Copy link
Contributor

Udokun commented Nov 25, 2024

что если начали 11, а потом зашло еще 50 - раунд все равно будет у всех без ограничений на скиллы).

Сам хотел написать про это, но посмотрел код и даже не стал предполагать, что у нас даже 10+ человек зайдут в раунд.

@Danistans
Copy link
Contributor

раундом, что если начали 11, а потом зашло еще 50 - раунд все равно будет у всех без ограничений на скиллы

Если к нам зайдёт 50 человек ради этого — пускай эта фишка будет заавтомержена

@DarthSidiousPalpatine
Copy link
Contributor

соглы, если 50 человек решили что хотят играть без скиллов и спецом заходят в лоупоп то справедливо будет замержить это

@Tap0r
Copy link
Contributor

Tap0r commented Nov 25, 2024

так там уже совмещённые профы, например медики замещают сразу весь медбей и имеют хорошие навыки во всех областях (даже не связанных с медициной)

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

@Tap0r
Copy link
Contributor

Tap0r commented Nov 25, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants