Add: [bounty] Voice update #6326

Pelmenik123 · 2025-01-06T00:30:43Z

Описание

Добавляет новый датум voice_model. Название - ложь. Он служит для запоминания и вспоминания голосов (и лиц)
rename_self добавлен новый аргумент mass_update, он отвечает за то какие отделы запомнят новое имя челикса. Всем профессиям был указан департамент. Поскольку были сложности с изменением имен боргов и ИИ они теперь тупо на вайтлисте (ну, имена всех не хуманов достпуны и так, без запоминания). (саботеры будут жить). Добавлен новый параметр в mind - special_role_meta_know. Если он true то ты будешь всегда знать имена людей с такой же спец ролью. Обновлонее голоса перенесено из Life в say (нужно будет хорошенько потестить)

Причина создания ПР

https://discord.com/channels/617003227182792704/755125334097133628/1308878958451752981

Демонстрация изменений

Изменения для смертных: теперь ты будешь знать голоса только коллег из отдела и голоса глав. Чтобы запомнить новые голоса достаточно шифткликнуть. Клоуны и мимов корректно знают после выбора имени

Тесты

Боже, оно работает?

Pelmenik123 · 2025-01-06T00:45:26Z

завтра дошлифую

into VOICE-UPDATE

BeebBeebBoob · 2025-01-08T04:11:27Z

ссылка не на #отчёты-по-предложениям.

Pelmenik123 · 2025-01-08T10:45:50Z

🤓

Antoonij · 2025-01-08T12:24:10Z

code/datums/datacore.dm

@@ -111,6 +114,7 @@ GLOBAL_LIST_EMPTY(PDA_Manifest)
 /datum/datacore/proc/manifest()
 	for(var/mob/living/carbon/human/H in GLOB.player_list)
 		manifest_inject(H)
+	SEND_GLOBAL_SIGNAL(COMSIG_SPECIAL_MASS_STORE_VOICE, GLOB.capitalist_manifest)


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

Вызов сигнала происходит в данном прока ибо данный прок отвечает за раундстарт заполнение манифеста что мне и нужно. Название такое поскольку это "уневирсальный" сигнал

Такое не нужно, нужно переименовать. Текущее название сигнала никак не связано с тем, что происходит в проке

так этот сигнал не специально для этого прока. Этот сигнал ну типо для массового запоминания по депортаментам

Все равно название менять под прок, а не под то, для чего он используется.

Система стиля говна

Antoonij · 2025-01-08T12:25:04Z

code/datums/voice.dm

+/datum/voice_model
+	var/mob/host = null //Носитель этой хуеты
+	var/tts_seed_string = "Arthas"
+	var/voice_gender = MALE
+	var/voice_name = "Неизвестный"
+	var/real_voice_name = "Неизвестный"
+
+	var/list/famous_voices = list()
+	var/list/famous_faces = list()


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

что за компоненты?

/datum/component

буээээ

Antoonij · 2025-01-08T12:26:27Z

code/__DEFINES/dcs/signals.dm

+//signal for mass roundstart store voice (very english)
+#define COMSIG_SPECIAL_MASS_STORE_VOICE "store_voice"
+
+#define COMSIG_JUST_LIST_ADD_VOICE "list_add_voice"


неиспользуемый сигнал со странным названием, just и list тут не нужны

Это забыл удалить

Antoonij · 2025-01-08T12:29:02Z

code/datums/datacore.dm

+		H.adv_voice.RegSignals()
+		H.UpdateVoice() //На всякий
+		var/prom_voice = H.adv_voice.voice_name 
+
+		var/datum/job/prom_job = SSjobs.GetJob(H.job)
+
+		for(var/dep_flag in prom_job.college_department) //:catsmile:
+			if(!isnull(GLOB.capitalist_manifest?[dep_flag]))
+				GLOB.capitalist_manifest[dep_flag][prom_voice] = H.real_name
+			else
+				GLOB.capitalist_manifest[dep_flag] = list((prom_voice) = (H.real_name))
+
+		//SEND_GLOBAL_SIGNAL(COMSIG_SPECIAL_MASS_STORE_VOICE, GLOB.capitalist_manifest)
+		//При создании рекордсов еще берется и голос, боже помилуй чтобы это ничего не сломало


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

убрать комментарии на русском языке

Регистрация идет на подсистему поскольку так работают глобальные сигналы. В не глобальных сигналах нет смысла. Я уберу коменты на русском кроме job.dm поскольку там важное предупреждение

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

к слову регистрация сигналов в манифесте происходит в целях оптимизации

посмотрю что такое компоненты, боже помилуй

Antoonij · 2025-01-08T12:30:10Z

code/datums/voice.dm

+	if(list_voice?["AbsolutePomny"])
+		famous_voices |= list_voice["AbsolutePomny"]


"AbsolutePomny" - это что? Нужно использовать нормальные названия ключей

это я забыл удалить

Antoonij · 2025-01-08T12:30:39Z

code/datums/voice.dm

+//Специфическая функция которая добавляет что нужно короче
+/datum/voice_model/proc/SpecialMassAddVoice(suka, list/list_voice)
+	SIGNAL_HANDLER
+	var/datum/job/prom_job = SSjobs.GetJob(host.job) //Нужно чтобы небыло предупеждений


убрать комментарии на русском языке/перевести их

Antoonij

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

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

Pelmenik123 · 2025-01-08T14:56:28Z

бляяяяяяяяяяять

Pelmenik123 · 2025-01-08T14:57:40Z

сидеть еще компоненты курить

BeebBeebBoob · 2025-01-08T17:40:30Z

На всякий поставил лейбл кандидата. Такой ПР на вид конфликтный т.к. существует вероятность что откатят такое.
Также костыли. Много чего подправить, а то переделать. Самое главное, нарушение стиля.
/proc/DoSomeItem()
Поменять на
/proc/do_some_item()

BeebBeebBoob · 2025-01-08T17:45:58Z

code/__DEFINES/is_helpers.dm

+#define isIdCard(A) (istype(A, /obj/item/card/id))
+
+#define isWallet(A) (istype(A, /obj/item/storage/wallet))


Suggested change

#define isIdCard(A) (istype(A, /obj/item/card/id))

#define isWallet(A) (istype(A, /obj/item/storage/wallet))

#define isidcard(A) (istype(A, /obj/item/card/id))

#define iswallet(A) (istype(A, /obj/item/storage/wallet))

Чем не нравится стиль с#

BeebBeebBoob · 2025-01-08T17:47:18Z

code/datums/voice.dm

@@ -0,0 +1,155 @@
+#define GENDER_NAME_UNKNOW list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL  = "Неизвестный")


Suggested change

#define GENDER_NAME_UNKNOW list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL = "Неизвестный")

#define GENDER_NAME_UNKNOWN list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL = "Неизвестный")

BeebBeebBoob · 2025-01-08T17:49:38Z

code/game/atoms.dm

@@ -1706,6 +1706,11 @@ GLOBAL_LIST_EMPTY(blood_splatter_icons)
 /atom/proc/GetTTSVoice()
 	return tts_seed

+/mob/proc/UpdateVoice()


Тут названия пороков решили на пофиг сделать.
Сохраняем стиль

Suggested change

/mob/proc/UpdateVoice()

/mob/proc/update_voice()

Это стиль с# был

BeebBeebBoob · 2025-01-08T17:51:04Z

code/datums/voice.dm

+
+	return get_gender_unknown_name(target.adv_voice.voice_gender)
+
+/datum/voice_model/proc/I_do_remember(mob/target)


Suggested change

/datum/voice_model/proc/I_do_remember(mob/target)

/datum/voice_model/proc/can_remember_voice(mob/target)

Не делаем, а можем ли сделать

Пупупу

BeebBeebBoob · 2025-01-08T17:52:22Z

code/datums/voice.dm

+	. = famous_voices?[target.adv_voice.voice_name]
+	if(.)


?!
Точка здесь тогда бессмысленна, если мы выводим либо TRUE или FALSE

Точка имеет смысл, иначе вывод неизвестный

BeebBeebBoob · 2025-01-08T18:06:36Z

code/modules/mob/mob.dm

+	voice_name = GetVoice()
+	adv_voice = new(src)


Мертвецам голос зачем??

Голос есть у всех мобов, но регистрация сигналов идёт только в daracore (экипаж) это нужно для корректной работы

Ладно, переставлю это в mob/living не учел эту залупу

BeebBeebBoob · 2025-01-08T18:08:01Z

code/modules/mob/mob_defines.dm

 	var/voice_name = "неизвестный голос"
+	var/datum/voice_model/adv_voice = null


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

Да, я переделаю это в компонент. Сука почему вы раньше не сказали так сдалать

BeebBeebBoob · 2025-01-08T18:09:28Z

code/modules/mob/mob_helpers.dm

@@ -577,9 +577,12 @@ GLOBAL_LIST_INIT(intents, list(INTENT_HELP,INTENT_DISARM,INTENT_GRAB,INTENT_HARM
 			length = length_char(oldname)
 			pos = findtextEx_char(objective.explanation_text, oldname)
 			objective.explanation_text = copytext_char(objective.explanation_text, 1, pos)+newname+copytext_char(objective.explanation_text, pos+length)
+	UpdateVoice()
+	if(LAZYLEN(mass_update) != 0)


Я думаю можно сделать и так

Suggested change

if(LAZYLEN(mass_update) != 0)

if(!LAZYLEN(mass_update))

Но я может немного туплю по спискам, но если список пустой то он и так 0, а значит FALSE а значит можно поменять с помощью !

Я сделал это поскольку не имею возможности быстро тестировать поведение бульена в таких моментах (загрузка по 5 минут :aga:)

Ну так то да, можно было и так

BeebBeebBoob · 2025-01-08T18:10:18Z

code/modules/mob/mob_say.dm

@@ -3,6 +3,7 @@
 	"\[" = "", "]" = "", "{" = "", "}" = "")

 /mob/proc/say(message, verb = "says", sanitize = TRUE, ignore_speech_problems = FALSE, ignore_atmospherics = FALSE, ignore_languages = FALSE)
+	UpdateVoice()


Мертвецам голос не нужен. Дайте живым. И да, пусть будет выдавать ошибку на это если дойдет до мертвецов

Мертвецы не говорят

BeebBeebBoob · 2025-01-08T18:12:42Z

code/datums/voice.dm

+#define GENDER_NAME_UNKNOW list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL  = "Неизвестный")
+#define FACE_MOD_SWITCH TRUE
+//Новая система голоса
+/datum/voice_model


Честно, это все можно было превратить в элемент или крайне компонент. Будет удобно и не засорять. Получение голоса можно связать по сигналу, а через проки и целый датум для каждого моба... Там ещё проки зачем-то в самом /mob
А можно было оставить для /mob/living ибо у мертвецов /mob/dead нету голоса

Всмысле living и dead, разве мертвый от живого не отличается только не работающим Life?

Я переделаю это в компонент как говорит фонд помощи, не вижу смысла в получении голоса по сигналу

Den109G · 2025-01-08T21:04:10Z

code/game/jobs/job/engineering.dm

@@ -1,3 +1,4 @@
+#define PRIMITIVE_STATION_DEPARTMENTS list("Command", "Medical", "Engineering", "Science", "Security", "Cargo", "Support", "Civilian")


А чем это отличается от station_departments? Вроде для того же самого используется.

Это используется так как GLOB.station_departaments иницилилизуруется во время запуска, примитив доступен для установки в качестве значения прямо так

Den109G · 2025-01-08T21:07:29Z

code/modules/mob/living/carbon/human/npcs.dm

 	equip_to_slot_if_possible(prom, ITEM_SLOT_CLOTH_INNER)
-
-	tts_seed = "Chen"
+	equip_to_slot_if_possible(punpun_id, ITEM_SLOT_ID)


Как Пун Пун связан с предложкой? И почему ему нужны часы за сервис?

Пун пуну не нужны часы за сервис. Я изменил немного пун пуна поскольку автор предложки давно просил отказаться от ошейника (убогая текстура на Монке)

Pelmenik123 · 2025-01-09T07:52:41Z

Костылей тут почти нет. Voice_model будет изменена на компоненты. Конфликтности не вижу на текущий момент (если новый сигнал фонда помощи уже в мердже то переделаю запоминание лиц)

Pelmenik123 added 5 commits December 1, 2024 19:49

ладно похуй уже

02328c7

микрофикс

1c44ae7

умер от кринжа

a18ef9f

костыльный фикс

e64f2c7

уу бляя

6116f77

github-actions bot added the Merge Conflict This doesn't seem right label Jan 6, 2025

Merge branch 'master220' into VOICE-UPDATE

494e2d7

github-actions bot removed the Merge Conflict This doesn't seem right label Jan 6, 2025

Pelmenik123 added 2 commits January 8, 2025 01:24

буэээ

c4a3473

Merge branch 'VOICE-UPDATE' of https://github.com/Pelmenik123/Pelmen-PRs

e8345a3

into VOICE-UPDATE

Pelmenik123 changed the title ~~Add: [bounty] (cringe) Voice update~~ Add: [bounty] Voice update Jan 7, 2025

Pelmenik123 added 2 commits January 8, 2025 02:28

bugfix

a39ddcd

nanofix

9b42ae5

BeebBeebBoob added the Требует отчёт предложки Всё ещё проходит голосовалки, либо ещё нету самой предложке в книге. label Jan 8, 2025

Antoonij reviewed Jan 8, 2025

View reviewed changes

пендосия апдейт

ae87b01

Antoonij reviewed Jan 8, 2025

View reviewed changes

Antoonij added the Do Not Merge Либо не прошла предложка, либо кто-то не доволен. label Jan 8, 2025

BeebBeebBoob requested changes Jan 8, 2025

View reviewed changes

Den109G reviewed Jan 8, 2025

View reviewed changes

		if(list_voice?["AbsolutePomny"])
		famous_voices \|= list_voice["AbsolutePomny"]

		#define isIdCard(A) (istype(A, /obj/item/card/id))

		#define isWallet(A) (istype(A, /obj/item/storage/wallet))

		@@ -0,0 +1,155 @@
		#define GENDER_NAME_UNKNOW list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL = "Неизвестный")

	#define GENDER_NAME_UNKNOW list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL = "Неизвестный")
	#define GENDER_NAME_UNKNOWN list(MALE = "Незнакомец", FEMALE = "Незнакомка", NEUTER = "Неизвестный", PLURAL = "Неизвестный")


		return get_gender_unknown_name(target.adv_voice.voice_gender)

		/datum/voice_model/proc/I_do_remember(mob/target)

	/datum/voice_model/proc/I_do_remember(mob/target)
	/datum/voice_model/proc/can_remember_voice(mob/target)

		var/voice_name = "неизвестный голос"
		var/datum/voice_model/adv_voice = null

		@@ -1,3 +1,4 @@
		#define PRIMITIVE_STATION_DEPARTMENTS list("Command", "Medical", "Engineering", "Science", "Security", "Cargo", "Support", "Civilian")

Add: [bounty] Voice update #6326

Are you sure you want to change the base?

Add: [bounty] Voice update #6326

Conversation

Pelmenik123 commented Jan 6, 2025 • edited Loading

Описание

Причина создания ПР

Демонстрация изменений

Тесты

Pelmenik123 commented Jan 6, 2025

BeebBeebBoob commented Jan 8, 2025

Pelmenik123 commented Jan 8, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Antoonij Jan 8, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Antoonij left a comment

Choose a reason for hiding this comment

Pelmenik123 commented Jan 8, 2025

Pelmenik123 commented Jan 8, 2025

BeebBeebBoob commented Jan 8, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Pelmenik123 commented Jan 9, 2025

Pelmenik123 commented Jan 6, 2025 •

edited

Loading

Antoonij Jan 8, 2025 •

edited

Loading