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

[Proposal] Удалить ex_act() атмосферных трубок #11104

Open
Deahaka opened this issue Mar 22, 2023 · 2 comments
Open

[Proposal] Удалить ex_act() атмосферных трубок #11104

Deahaka opened this issue Mar 22, 2023 · 2 comments
Labels

Comments

@Deahaka
Copy link
Contributor

Deahaka commented Mar 22, 2023

Подробное описание проблемы

Слева до, справа после по профайлеру
00000000000000000000000000000000003

/obj/machinery/atmospherics/ex_act(severity)

Оно експлоудит атомы внутри себя, но они и без этого гибаются если оказываются в DEVASTATE радиусе. Если нужен не только гиб, то что ещё?

Что должно было произойти

нет лишних задач во время взрыва нюки

Что произошло на самом деле

есть лишние задачи во время взрыва нюки

Как повторить

затестить #11090

Дополнительная информация:

@LudwigVonChesterfield
Copy link
Contributor

в рнд расписал тебе полное решение

@volas
Copy link
Member

volas commented Mar 23, 2023

Оно експлоудит атомы внутри себя, но они и без этого гибаются если оказываются в DEVASTATE радиусе

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

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

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

No branches or pull requests

4 participants