Holochips: Return by Chip11-n · Pull Request #9784 · TauCetiStation/TauCetiClassic

Chip11-n · 2022-08-06T13:54:45Z

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

Надеюсь я не свалю куда подальше. И так...

У ЕРТ (ОБР), нюкеров, дедсадовцев, воксов появляются голографические карты.

Само по себе это чип, вставляемый в шлем (именно шлем), и вытаскиваемый из него отвёрткой. Все, кто находятся на станции, имеют одинаковую частоту и шифровку (на подобии сигналлеров), отображаются вам на карте. Частот 400 (нечётные и чётные - для увеличения кол-ва комбинаций, оставляя возможность "украсть" частоту), шифровки ещё 1000. В каждом раунде частоты генерируются рандомно.

Голочип

Голокарта

Аналог у дедсадовцев

Неустанные напоминания

#9762 (comment)
https://discord.com/channels/401030271417188353/401031448963973120/1003319427636404417
https://discord.com/channels/401030271417188353/401031448963973120/1000755363756380371

Уже самому смешно.

Теперь смешно вдвойне

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

Авторство

Originally somewhere from /VG/

Чеинжлог

🆑 Filatelele && Chip11-n

add[link]: Голочипы!

Mostly copypaste of old mechanics from gamma, small rework of how holochips are displayed on the map.

Holochips are now actually spawn in ERT, syndi and deathsquad helmets.

Removed leftovers from old iterations of holomap system

Holochips can be crafted via protolathe, price is probably to be changed

Leftover code prevented compiling. Also removed some faulty copypaste.

Lowercased HEX codes

…e/TauCetiClassic into holomap-from-the-crypt

Playsound is now cool

Changed proc naming to more clear, no world.log now

Instead of 0

sanitiye_frequency sanitized in wrong boundaries.

… removed excessive return

volas · 2022-09-03T10:51:36Z

code/controllers/subsystem/holomaps.dm

+#define HOLOMAP_WALKABLE_TILE "#66666699"
+#define HOLOMAP_CONCRETE_TILE "#FFFFFFDD"
+
+/datum/controller/subsystem/holomaps/proc/generate_holo_map()


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

KIBORG04 · 2022-09-04T19:15:42Z

смешные рантаймы))

volas · 2022-09-05T19:33:06Z

code/controllers/subsystem/holomaps.dm

+			if(!marker_location)
+				stack_trace("[HC.holder]/[HC.holder.loc]/[HC.frequency] without turf.")


312 Runtime in unsorted.dm:1525 : The emergency response team commander helmet//1438 without turf. 247 Runtime in unsorted.dm:1525 : The alien visor//1483 without turf. 161 Runtime in unsorted.dm:1525 : The deathsquad helmet//1386 without turf. 52 Runtime in unsorted.dm:1525 : The emergency response team security helmet//1406 without turf.

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

оно всё с мастера,

[19:32:24] Runtime in unsorted.dm:1525 : The emergency response team commander helmet//1438 without turf. proc name: stack trace (/proc/stack_trace) src: null call stack: stack trace("The emergency response team co...") Holomaps (/datum/controller/subsystem/holomaps): process holomap markers() Holomaps (/datum/controller/subsystem/holomaps): fire(0) Holomaps (/datum/controller/subsystem/holomaps): ignite(0) Master (/datum/controller/master): RunQueue() Master (/datum/controller/master): Loop() Master (/datum/controller/master): StartProcessing(0)

наобум - может зависшие в дестрое итемы.

volas · 2022-09-05T19:36:41Z

code/controllers/subsystem/holomaps.dm

+			if(!I)
+				stack_trace("[HC.type]/[HC.holder.loc]/[HC.frequency] failed to generate image.")


261 Runtime in unsorted.dm:1525 : /obj/item/holochip/M'ark M'rshan/1400 failed to generate image. 50 Runtime in unsorted.dm:1525 : /obj/item/holochip/Andre Isaakov/1400 failed to generate image.

volas · 2022-09-14T19:03:02Z

code/modules/holomap/holochip.dm

+/obj/item/holochip/proc/deactivate_holomap()
+	if(!activator)
+		return
+	activator.holomap_obj.cut_overlay(holomap_base)


Runtime in holochip.dm:71 : Cannot execute null.cut overlay()

стека нет

volas · 2022-09-23T02:34:26Z

code/controllers/subsystem/holomaps.dm

+/datum/controller/subsystem/holomaps/proc/process_holomap_markers()
+	for(var/freq in SSholomaps.holochips)
+		for(var/obj/item/holochip/HC in SSholomaps.holochips[freq])
+			var/turf/marker_location = get_turf(HC)


Suggested change

var/turf/marker_location = get_turf(HC)

var/turf/marker_location = get_turf(HC.holder)

я тут посмотрел и заметил, что чипы похоже не лежат физически в шлеме, и шлем с чипом просто перелинкиваются через свои ссылки. Должно исправить ошибку.

volas · 2022-09-23T02:38:32Z

code/modules/holomap/holochip.dm

@@ -0,0 +1,218 @@
+#define OFFSET_CORRECTOR 6
+/obj/item/holochip


А никто не говорил еще слово на букву "к"? Впрочем, опционально

volas · 2022-10-01T13:08:12Z

если вечером не увижу их в рантаймах и куделях, то мержу

Chip11-n · 2022-10-01T15:35:08Z

Они есть ещё в рантаймах, но продвижения также есть

Sakuya-Izayoi · 2022-10-02T12:44:07Z

code/modules/clothing/head/helmet.dm

+	if(holochip)
+		holochip.remove_action(user)
+		holochip.deactivate_holomap()
+		if(SSholomaps.holochips[num2text(holochip.frequency)] && (holochip in SSholomaps.holochips[num2text(holochip.frequency)]))


огромное нагромождение операторов. нужно сделать прок который в выполняет вот это все .holochips[num2text(...)]

Почему вообще везде обязателен num2text, и почему его нужно делать на каждой строчке по несколько раз. Не проще будет frequency сразу делать текстом?

И вообще стоило бы не изобретать частоты еще раз, а использовать уже существующие /datum/radio_frequency/

В существующих частотах много для меня нагромождений, мне же нужен только список в списке.
С текстом я определённо еще гляну

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

Chip11-n · 2022-10-10T13:47:25Z

Ну и что тебе надо, бот?

volas · 2022-10-14T05:35:49Z

code/modules/holomap/holochip.dm

+	holomap_base = SSholomaps.default_holomap
+	if(color_filter)
+		holomap_base.color = color_filter
+	activator.holomap_obj.add_overlay(holomap_base)


2 Runtime in holochip.dm:60 : Cannot execute null.add overlay().

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

Посмотри сегодня-завтра и я уже мержу. Иначе придется ждать уже 4 попытку, с чипами на компонентах и контроллере.

Иногда этого оверлея не создаётся, потому и простыми вопросиками фиксится.

А почему не создается? Потому что у активатора нет худа голомапы. А почему нет худа голомапы?

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

Filatelele added 30 commits December 7, 2021 23:02

Init

cfd46a9

Mostly copypaste of old mechanics from gamma, small rework of how holochips are displayed on the map.

Holochips in helmets

34813b0

Holochips are now actually spawn in ERT, syndi and deathsquad helmets.

Removed garbage

1287766

Removed leftovers from old iterations of holomap system

Added holochips to rnd

0562688

Holochips can be crafted via protolathe, price is probably to be changed

What a shame

f4c1b19

Leftover code prevented compiling. Also removed some faulty copypaste.

Merge branch 'master' into holomap-from-the-crypt

cb659f2

HEX lowercase

9327d44

Lowercased HEX codes

Merge branch 'holomap-from-the-crypt' of https://github.com/Filatelel…

5fbff71

…e/TauCetiClassic into holomap-from-the-crypt

Playsound

8c11895

Playsound is now cool

Now naming is obvious

dbd981e

Changed proc naming to more clear, no world.log now

Proc naming change and to_chat changed from third person style

1813a59

MOUSE_OPACITY_TRANSPARENT

698458a

Instead of 0

Added define ADD_TO_GLOBAL_LIST

7ad86df

Added global. to addresing global variables

9bad201

Added holochip design to RnD technologies

635cafa

Fixed errors with frequency selection

0325102

sanitiye_frequency sanitized in wrong boundaries.

Fixed holomarker drift

321c74f

Removed unnecessary holders

12bd138

Correctly declared global variable

84b9d52

user.drop_from_inventory(I, src)

ab51584

Transitioned holochip holder to clothing/head level

3509ba2

It was intended to be here

4005fb5

Reverted removal of holders in atom_init + added AB_CHECK_ALIVE flag

3bbb761

Moved activation from holochip to datum

f130928

Caching rework, added shuttle markers, added vox holochips

c429ade

Fixed brainlag

ed36f7f

Removed leftovers

9464797

Added check for abstract items

ae54832

Changed 16 to #define & every mob/living/carbon is shown on holomap &…

37afaf3

… removed excessive return

Added SSholomaps

529be15

volas reviewed Sep 3, 2022

View reviewed changes

volas reviewed Sep 5, 2022

View reviewed changes

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

allign

c411a47

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

volas reviewed Sep 14, 2022

View reviewed changes

volas reviewed Sep 23, 2022

View reviewed changes

volas and others added 7 commits September 23, 2022 05:39

Update code/controllers/subsystem/holomaps.dm

e670054

state

c42bff4

Merge remote-tracking branch 'upstream/master' into Holochips

e61be61

Update actions.dmi

9040953

testing on nukies

d109cd2

Update helmet.dm

5eb344b

Update holochip.dm

7c0fc6f

Chip11-n and others added 2 commits October 1, 2022 18:41

Update holochip.dm

8913ec2

Update helmet.dm

7db14b4

Sakuya-Izayoi reviewed Oct 2, 2022

View reviewed changes

freqs

a7caa59

github-actions bot added the Merge Conflict label Oct 8, 2022

Merge remote-tracking branch 'upstream/master' into Holochips

78a4490

github-actions bot removed the Merge Conflict label Oct 13, 2022

volas reviewed Oct 14, 2022

View reviewed changes

no_need_on_npc

cf21cbd

volas merged commit 787e546 into TauCetiStation:master Oct 16, 2022

TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Oct 16, 2022

Automatic changelog generation for PR TauCetiStation/TauCetiClassic#9784

74c8f98

		if(!marker_location)
		stack_trace("[HC.holder]/[HC.holder.loc]/[HC.frequency] without turf.")

		if(!I)
		stack_trace("[HC.type]/[HC.holder.loc]/[HC.frequency] failed to generate image.")

	var/turf/marker_location = get_turf(HC)
	var/turf/marker_location = get_turf(HC.holder)

		@@ -0,0 +1,218 @@
		#define OFFSET_CORRECTOR 6
		/obj/item/holochip

Uh oh!

Conversation

Chip11-n commented Aug 6, 2022

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

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

Авторство

Чеинжлог

Uh oh!

Choose a reason for hiding this comment

Uh oh!

KIBORG04 commented Sep 4, 2022

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

volas commented Oct 1, 2022

Uh oh!

Chip11-n commented Oct 1, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Chip11-n Oct 4, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Chip11-n commented Oct 10, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

Chip11-n Oct 15, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

Chip11-n Oct 4, 2022 •

edited

Loading

Chip11-n commented Oct 10, 2022 •

edited

Loading

Chip11-n Oct 15, 2022 •

edited

Loading