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

Atmos Threshold configued #2106

Merged
merged 2 commits into from
Oct 18, 2024
Merged

Conversation

UrPrice
Copy link
Collaborator

@UrPrice UrPrice commented Oct 17, 2024

Описание PR

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

Перед этим я достаточно ознакомился с:
space-wizards#29504
space-wizards#28370
space-wizards#29493

И изучил причины введения всей системы, согласно: https://docs.spacestation14.com/en/space-station-14/departments/atmos/proposals/atmos-rework.html

Я считаю, что у Partmedia данный функционал всё ещё в экспериментальном виде и предлагаю снизить порог до момента пока либо:

  1. не соберём метрики, позволяющее определить как часто атмосферные техники взаимодействуют с воздушными сигналками по станции
  2. PartMedia и энтузиасты не отконфигурируют этот порог самостоятельно

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

Процесс разгерметизации занимает действительно время, чтобы все моли воздуха ушли, однако они уходят равномерно и постепенно, если справедливо условие, что присутствует источник разгерметизации. Также и смешивание газов комнат, одна из которых содержит условие разегрметизации - лишь определяет соседнюю комнату источником, из-за чего не происходит резких смешиваний. Однако как только пожарные шлюзы перекрывают атмосферные участки (комнаты/коридоры) - комната, потерявшая N количество воздуха при смешивании с другой комнатой - производит это резко, просто уравнивая свойства газов двух комнат. Соответственно это происходит при условии, что источника разгерметизации нет.

Из этой проблематики, представим зал прибытия в котором образовалась дыра от метеорита. Прибывший игрок пройдет через рагерметизацию, спустит в соседней комнате воздух. После начнёт идти далее по коридорам внутрь станции, попутно создавая резкие перепады давления в комнатах. В нашей ситуации, с порогом в 80 это давление из-за перепадов очень быстро снижается до порога - когда давление не восстанавливается самостоятельно. Учитывая наши большие комнаты, восстановить его становится ещё сложнее. В такой ситуации Атмосферный Техник становится нужен сразу во всех частях станции, ведь место разгерметизации в основном ни только в зале прибытия.

С порогом в 40, мы расстянем момент утечки блокировки за счёт не такой близкой дистанции между дефолтом и порогом (110 - 80 с разницой в 30 или 110-40 с разницой в 70). Убийство давлением по кукле (человеку) начинается, если порог ниже 50. То есть будет ещё расстояние в 10кПа, которые помогут атмос-техникам восстановить атмосферу, не доведя её до заблокированной и убийственной.

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

Медиа
Отсутствует.

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения
🆑

  • tweak: изменён порог блокировки воздушных вентиляций с 80кПа до 40кПа!

@UrPrice UrPrice requested a review from Ady4ik October 17, 2024 22:43
@UrPrice UrPrice changed the title SS220 Atmos Threshold configued Atmos Threshold configued Oct 17, 2024
Copy link
Collaborator

@stalengd stalengd left a comment

Choose a reason for hiding this comment

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

Мало ведаешь

…ent.cs

Co-authored-by: Stalen <33173619+stalengd@users.noreply.github.com>
@UrPrice UrPrice requested review from stalengd and removed request for stalengd October 17, 2024 22:55
@stalengd stalengd removed their request for review October 17, 2024 22:56
@stalengd stalengd merged commit 16906cb into SerbiaStrong-220:master Oct 18, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants