Telespace jammer by volas · Pull Request #9659 · TauCetiStation/TauCetiClassic

volas · 2022-07-18T04:12:16Z

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

Добавляет на станцию генератор помех для технологий телепортации.

Прокручивающаяся к полу хреновина с радиусом действия от 1 до 5. Не позволяет кому-либо телепортироваться в зону действия (из зоны - можно, если это не ethereal jaunt).

Для настройки (мультитулом) и переноса (откручиваем ключем) требует доступ главы для разблокировки.

На боксе защищены комната ИИ на спутнике, капитанская, оружейная СБ (обе), ваулт и ЕВА.

Resolve #7083

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

Телепорты сейчас - имба без контры. Антагонисты без проблем проникают в самые защищенные отсеки станции, телесаентисты троллят малфов и капитанов.

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

Авторство

Спрайт - народное творчество.

Чеинжлог

🆑

add[link]: Генераторы помех для технологий телепортации. Закинуты в некоторые отсеки на боксе.

TauKitty · 2022-07-18T04:12:18Z

Changelog status: ✔️

volas · 2022-07-18T04:20:12Z

 #define PASSMOB     32

 //turf-only flags
-#define NOJAUNT			1


NOJAUNT - не работающее легаси. Когда-то предполагалось, что святая вода на турфе будет блокировать телепорты культистов (вроде), но со временем этот функционал куда-то потерялся и остался только флаг.

volas · 2022-07-18T04:22:25Z

+			update_affected_zone()
+			return
+
+/obj/machinery/telescience_jammer/emag_act(mob/user)


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

Еще я хотел добавить отключение от ЭМИ для мага, забыл.

Еще я хотел добавить отключение от ЭМИ для мага, забыл.

Конкретно для ЭМИ мага или для любого ЭМИ?

(Под самое удаление малфа и после этого, людям начало приходить в голову, что — закинуть около ядра хотя бы 150/150 железо/уран, уже не плохо так подпортит жизнь малфу. Не убьет, но при нескольких таких гранатах, может разрядить ближайшие АПЦ и оставить самого ИИ на 1% хп. Если постараться. Учитывая что какой-либо защиты от распространения ЭМИ, или же экранирования — нет, ситуация с закидывающим под нос ИИ лимитку ученым не особо сильно изменится, просто добавится факт того, что нужно будет открыть воронку сначала рядом со спутником, кинуть ЭМИ, после этого уже открывать воронку на сам спутник спокойно кидая бомбу.)

Deahaka · 2022-07-18T05:05:57Z

Запрет лёгкого раундстарт лута кэпской механом — круто.

simb11 · 2022-07-18T06:28:45Z

А в ЕВА зачем эта штука? В ЕВА нет ничего прям супер важного , да и доступ туда есть почти у каждого члена экипажа.

notkripton · 2022-07-18T06:49:31Z

Спутник весь блокается или только хижина ии?

Mercurialaste · 2022-07-18T06:51:37Z

Спутник весь блокается или только хижина ии?

Только хижина, максимальный радиус 5 тайлов…
(По крайней мере, как в понимаю под комнаткой на спутнике ИИ подразумеваются не телекомы/рядом с телекомами, раз речь заходила про малфа и лимитки.)

Sakuya-Izayoi

у дропподов синдиката есть свой механ на запрет дропа в определенную область. стоит объединить.

Sakuya-Izayoi · 2022-07-18T09:27:53Z

+	var/turf/T = parent
+	//T.color = "#ff0000"


Suggested change

var/turf/T = parent

//T.color = "#ff0000"

тайпкаст уберу, коммент останется мапперам в помощь

может тогда лучше дебаг верб "подсвечивать тайлы в которые нельзя телепортироваться"?

верб не стоит времени, которое уйдет на него. Если коммент режет глаза - могу убрать

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

Sakuya-Izayoi · 2022-07-18T09:34:52Z

+	layer = TURF_LAYER+0.1
+	plane = FLOOR_PLANE


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

LudwigVonChesterfield · 2022-07-18T15:24:43Z

Выходит можно выпилить фигню что джаунт мага в стену гибает?

volas · 2022-07-18T16:30:54Z

Выходит можно выпилить фигню что джаунт мага в стену гибает?

С ними там еще другая проблема, мы технически не должны позволять мобам оказываться в тайлах с density = TRUE, а то они становятся неуязвимыми и вообще это баг.

А в ЕВА зачем эта штука? В ЕВА нет ничего прям супер важного , да и доступ туда есть почти у каждого члена экипажа.

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

Спрайт мне дали, завтра обновлю и поставлю в ТМ.

Co-authored-by: Sakuya-Izayoi <30875884+Sakuya-Izayoi@users.noreply.github.com>

volas · 2022-07-21T02:04:54Z

Мерж реди.

Для дальнейших улучшений, вне данного PR-а:

Реализовать COMPONENT_INTERFERE_TELEPORT. В некоторых случаях можно не просто пресекать телепорт, но и выталкивать за пределы поля или еще как-то по особому мешать. Это должен поддерживаться со стороны телепорта, а не компоненты.
Перенять идею с "каналами" телепортации с тг. Червоточины - свой тип технологии, телепортации - свой, и так далее. Так возможно вернуть освящение тайлов священнику (NOJAUNT), которое будет работать против телепортаций демонических антагов, но не против обычных.
Стоит отрефакторить код телепортации. Их несколько видов, каждый со своим кодом, можно местами обобщить.

Sakuya-Izayoi · 2022-07-21T21:08:09Z

не вижу блокировки дроп подов, хотя ты даже поставил лайк мне в #7083 (comment)

volas · 2022-07-22T06:48:58Z

дропподы падают с неба, а не телепортируются, пусть будет их фичей

Sakuya-Izayoi · 2022-07-22T18:08:36Z

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

Sakuya-Izayoi

spellcheck

NinjaPikachuska · 2022-07-22T18:33:07Z

+		return
+	if(prob(80/severity))
+		stat |= EMPED
+		addtimer(CALLBACK(src, .proc/after_emp), 10 MINUTES / severity)


Тут разве не должен быть апдейт_айкон?

иконка для EMPED не изменяется, нет

Хотя, отчасти да, из-за is_operational emped будет изменять иконку. Я так не хотел, поправлю.

Co-authored-by: Sakuya-Izayoi <30875884+Sakuya-Izayoi@users.noreply.github.com>

volas · 2022-07-23T01:16:18Z

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

Если рядом будет свободный тайл - у него еще есть шанс. Или если кто-то отключит устройство в это время.

telespace jammer

1c338d6

TauKitty added Map Edit Sprites labels Jul 18, 2022

This comment was marked as outdated.

Sign in to view

volas commented Jul 18, 2022

View reviewed changes

volas added the Feature Новая фича label Jul 18, 2022

Sakuya-Izayoi reviewed Jul 18, 2022

View reviewed changes

Sakuya-Izayoi added the Sprite Needs label Jul 18, 2022

Sakuya-Izayoi removed the Sprite Needs label Jul 18, 2022

volas and others added 2 commits July 19, 2022 16:42

Apply suggestions from code review

4207c4b

Co-authored-by: Sakuya-Izayoi <30875884+Sakuya-Izayoi@users.noreply.github.com>

New icon, emp act

5705416

volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jul 19, 2022

no avoiding through wall ejecting

94ab9ba

volas requested a review from Sakuya-Izayoi July 21, 2022 01:56

Sakuya-Izayoi reviewed Jul 22, 2022

View reviewed changes

Comment thread code/game/machinery/jammer.dm Outdated

Comment thread code/game/machinery/jammer.dm Outdated

Comment thread code/game/machinery/jammer.dm Outdated

Comment thread code/game/machinery/jammer.dm Outdated

NinjaPikachuska reviewed Jul 22, 2022

View reviewed changes

Apply suggestions from code review

da23dec

Co-authored-by: Sakuya-Izayoi <30875884+Sakuya-Izayoi@users.noreply.github.com>

volas commented Jul 23, 2022

View reviewed changes

Comment thread code/game/machinery/jammer.dm Outdated

volas and others added 2 commits July 23, 2022 04:22

fix icon update

609846c

Merge branch 'master' into jammer

8c57268

Sakuya-Izayoi merged commit a0bc97c into TauCetiStation:master Jul 23, 2022

TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Jul 23, 2022

Automatic changelog generation for PR TauCetiStation/TauCetiClassic#9659

115989e

Uh oh!

Uh oh!

Conversation

volas commented Jul 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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

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

Авторство

Чеинжлог

Uh oh!

TauKitty commented Jul 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changelog status: ✔️

Uh oh!

This comment was marked as outdated.

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Deahaka commented Jul 18, 2022

Uh oh!

simb11 commented Jul 18, 2022

Uh oh!

notkripton commented Jul 18, 2022

Uh oh!

Mercurialaste commented Jul 18, 2022

Uh oh!

Sakuya-Izayoi left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LudwigVonChesterfield commented Jul 18, 2022

Uh oh!

volas commented Jul 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

volas commented Jul 21, 2022

Uh oh!

Sakuya-Izayoi commented Jul 21, 2022

Uh oh!

volas commented Jul 22, 2022

Uh oh!

Sakuya-Izayoi commented Jul 22, 2022

Uh oh!

Sakuya-Izayoi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

volas commented Jul 23, 2022

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

volas commented Jul 18, 2022 •

edited

Loading

TauKitty commented Jul 18, 2022 •

edited

Loading

volas commented Jul 18, 2022 •

edited

Loading