A bomb! Взрывы которые (вероятно) не будут смержены #11090
Conversation
Changelog status: ✔️ |
всё таки не лишний видимо
Deahaka
changed the title
|
В маленьком радиусе классно и прикольно. В большом плохо и всё умирает. Рекурсивный взрыв намного оптимизированнее смотрится чем подсистемный. |
|
tgstation/tgstation#50594 не очень атомарен и там много вещей и небольших оптимизаций. По совпадению, часть с тиками я как раз взял сюда #11139 Из самого важного остаётся крайне затратный ребилд труб, про которые ты сам указал в ишью, но их можно портировать отдельно. Я ок с тем, что бы этот PR был только про сабсистему, единая сортируемая очередь со своим приоритетом уже прогресс. Но можешь еще портировать небольшие оптимизации вроде https://github.com/tgstation/tgstation/pull/50594/files#diff-ffa07d053aa8e243f3c188ffc6632ab0a6c0cab71357cfa3619af7549c2ca515R492-R500 - где вместо рекурсивного вызова ex_act "прям щас", вещи добавляются в очередь и взрываются по расписанию контроллера. |
|
всё что нашёл в поисковике по поводу |
|
|
||
| return outlist | ||
|
|
||
| /proc/propagate_blastwave(atom/epicenter, devastation_range, heavy_impact_range, light_impact_range, flash_range) |
There was a problem hiding this comment.
перенеси в контроллер и поправь пожалуйста порядок проков, дифтул с ума сходит если пытаться сверять с тг.
Еще сразу стоит перенести /explode в контроллер, где он там сейчас лежит.
There was a problem hiding this comment.
ну и комментарии тоже не стоит оставлять за бортом, раз их кто-то уже написал и ты всё равно переносишь
There was a problem hiding this comment.
сделал порядок проков как на ТГ, перенёс всё как там в подсистему. Комментарии разместил рядом, хотя я не вижу в них чего-то полезного.
| #define EX_ACT(target, args...)\ | ||
| target.ex_act(##args); | ||
|
|
||
| SUBSYSTEM_DEF(explosions) |
There was a problem hiding this comment.
на тг еще удобный /stat_entry со стоимостью взрывов и состоянием сабсистемы, была ли причина не переносить?
There was a problem hiding this comment.
я просто не знаю что это такое. Добавил
|
Спасибо за старт задачи, закрою в пользу #11256 |
Описание изменений
Только для теста на сендбоксе
Почему и что этот ПР улучшит
взрывы мгновенные. Но только не с нюкой.
Авторство
TG, tgstation/tgstation#50594
Чеинжлог
🆑