Даём возможность пощупать органы и вырывать их #12653

maleyvich · 2023-12-29T10:31:04Z

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

добавление спрайтов для органов, создание операции для вытаскивания органов и их вставки

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

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

Авторство

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

Чеинжлог

🆑

image: Были добавлены спрайты для органов.
rscadd: Была дополнена операция на вынимание имплантов: теперь можно вынимать и сами органы пациента
tweak: Операция по выниманию мозга была обьединена с операцией на вынимание органов и немного усложнена.
map: В холодильниках операционных, на склад меда и робототехники теперь находятся органы.

This reverts commit 082e103.

TauKitty · 2023-12-29T10:31:07Z

Changelog status: ✔️

maleyvich · 2023-12-29T11:00:55Z

ингейм отображение сердец (проблема - обычное человеческое сердце очень маленькое)

UDaV73rus · 2023-12-29T11:04:04Z

Они такие маленькие на других билдах, чтобы когда устраиваешь резню, весь коридор не был забит органами размером с весь тайл. А у нас есть ворлд иконки и норм

maleyvich · 2023-12-29T11:26:07Z

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

Chip11-n · 2023-12-29T11:46:33Z

Пластиковым выглядит сердце. Может всё-таки запрещенный-красный?

maleyvich · 2023-12-29T11:47:16Z

Пластиковым выглядит сердце. Может всё-таки запрещенный-красный?

поспрашивал игроков в етолксе - им понравилось, хз, уже на мнение спрайтдепа

KIBORG04 · 2023-12-29T12:43:58Z

Ты в курсе, что спрайтов добавить мало? Там надо много кода рефакторить и заставить его работать. Сейчас органов даже в кукле физически нету, они в нуллспейсе лежат и через него работают

maleyvich · 2023-12-29T12:44:37Z

Ты в курсе, что спрайтов добавить мало? Там надо много кода рефакторить и заставить его работать. Сейчас органов даже в кукле физически нету, они в нуллспейсе лежат и через него работают

в курсе, поэтому и WiP

maleyvich · 2023-12-29T13:03:46Z

были добавлены легкие всем (кроме дион), пока всё суммарно выглядит вот так

Deahaka · 2023-12-29T13:23:22Z

Как это улучшит РП?

NinjaPikachuska · 2023-12-29T13:25:07Z

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

maleyvich · 2023-12-30T07:45:55Z

я не понимаю что я наделал

KandreyKandrey · 2024-01-03T16:53:28Z

code/modules/organs/organ_internal.dm

@@ -6,6 +6,8 @@

 	// Strings.
 	var/organ_tag   = null      // Unique identifier.
+	var/list/organ_races =  list() // for species (human, tajaran, vox, etc.)


а зачем этот список то?
(он сейчас нигде не используется)

я попросту задал переменную в виде листа, ниже organ_races используется для органов

другой вопрос правильно ли я все сделал

code/modules/organs/organ.dm

MrPerenka · 2024-01-03T21:33:58Z

Я что-то так посмотрел, и толи я не умею читать код, толи тут не прописан способ получения органов. Так где их брать? Карго, роботехника, авто/протолат, ксенобио, раундстарт?

maleyvich · 2024-01-04T02:50:03Z

Я что-то так посмотрел, и толи я не умею читать код, толи тут не прописан способ получения органов. Так где их брать? Карго, роботехника, авто/протолат, ксенобио, раундстарт?

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

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

…assic into heartsprites

maleyvich · 2024-01-04T04:18:41Z

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

code/modules/surgery/implant.dm

NinjaPikachuska

о, да, еще забыл упомянуть важную вещь!
в билде полно мест где НЕ подразумевается отсутствие органов (в частности глаз)
и нужно пройтись по этим местам

code/modules/mob/living/carbon/brain/MMI.dm

NinjaPikachuska · 2024-01-05T16:18:22Z

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

+		//Liver
+		if(should_have_organ(O_LIVER) && !has_organ(O_LIVER))
+			adjustToxLoss(0.2)
+			blurEyes(0.5)
+
+		//Kidneys
+		if(should_have_organ(O_KIDNEYS) && !has_organ(O_KIDNEYS))
+			adjustToxLoss(0.4)
+			adjustBruteLoss(0.2)
+
+		//Heart
+		if(should_have_organ(O_HEART) && !has_organ(O_HEART))
+			death()
+
+		//Lungs
+		if(should_have_organ(O_LUNGS) && !has_organ(O_LUNGS))
+			adjustOxyLoss(1.5)


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

NinjaPikachuska · 2024-01-05T16:18:34Z

code/modules/organs/organ.dm

 /obj/item/organ/proc/insert_organ(mob/living/carbon/human/H, surgically = FALSE, datum/species/S)
 	set_owner(H, S)

-	STOP_PROCESSING(SSobj, src)


зачем?

меня попросили я и убрал это, уверяя что процессинг выполняется самим хуманом

NinjaPikachuska · 2024-01-05T16:19:38Z

code/modules/organs/organ.dm

@@ -34,14 +34,23 @@
 	loc = null
 	owner = H

+/obj/item/organ/proc/del_owner(mob/living/carbon/human/H)
+	loc = null
+	owner = null


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

чтобы ТОЧНО решить проблему что орган остаётся в теле даже если его достанут

NinjaPikachuska · 2024-01-05T16:21:24Z

code/modules/organs/organ.dm

 	if(parent_bodypart)
 		parent = owner.bodyparts_by_name[parent_bodypart]

+
+/obj/item/organ/proc/remove_organ(mob/living/carbon/human/H, surgically = FALSE, datum/species/S)
+	del_owner(H, S)


тут даже аргументы не совпадают с тем что объявлено в проке 😔

code/modules/surgery/bodypart_manipulation.dm

NinjaPikachuska · 2024-01-05T17:24:42Z

code/modules/surgery/bodypart_manipulation.dm

+					var/mob/living/simple_animal/borer/borer = target.has_brain_worms()
+
+					if(borer)
+						borer.detatch() //Should remove borer if the brain is removed - RR
+					remove_from_cavity(user, target, choosen_object, BP, tool)


эээ здесь тоже что-то нездоровое почему мы вытаскиваем борера вне проверки на мозг

NinjaPikachuska · 2024-01-05T17:25:52Z

code/modules/surgery/bodypart_manipulation.dm

+					if(istype(choosen_object, /obj/item/organ/internal))
+						var/obj/item/organ/internal/H = choosen_object
+						BP.bodypart_organs -= choosen_object
+						H.remove_organ(choosen_object)
+						H.loc = get_turf(target)
+					else if (istype(choosen_object, /obj/item/organ/internal/brain))


никогда не дойдет этой проверки, брейн и есть /organ/internal

гитхаб написал outdated но оно осталось неизмененным 👴

code/modules/surgery/bodypart_manipulation.dm

code/modules/surgery/braincore.dm

Co-authored-by: NinjaPikachuska <89906909+NinjaPikachuska@users.noreply.github.com>

4310V343k · 2024-01-07T04:40:10Z

code/modules/surgery/bodypart_manipulation.dm

 		var/list/embed_object_else = list()
-		for(var/embed_object in BP.implants)
+		for(var/embed_object in BP.implants || BP.bodypart_organs)


это так не работает

однако, все таки без этого органы не будут показываться в радиалке.

да что ты такое говоришь

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

for(var/el in list1 || list2) итерирует только по list1

4310V343k · 2024-01-07T04:41:16Z

code/modules/surgery/bodypart_manipulation.dm

 			if(istype(embed_object, /obj/item/weapon/shard/shrapnel))
 				embed_object_shrapnel += embed_object
 				continue
 			if(istype(embed_object, /obj/item/weapon/implant))
 				embed_object_implants += embed_object
 				continue
+			if(istype(embed_object, /obj/item/organ/internal))
+				BP.bodypart_organs.Copy()


зачем эта копия? может она должна была быть строкой ниже?

оно работает именно если копию сделать сперва, а не после

копия, которую ты делаешь, нигде не сохраняется и ни на что не влияет

4310V343k · 2024-01-07T04:42:40Z

code/modules/surgery/bodypart_manipulation.dm

+						BP.bodypart_organs -= choosen_object
+						H.remove_organ(choosen_object)
+						H.loc = get_turf(target)
+					else if(istype(choosen_object, /obj/item/organ/internal/brain))


код под этой ифкой недосягаем

я уже пытался её как угодно дергать, не получается однако...

просто сделай это как обычную ифку?

4310V343k · 2024-01-07T04:46:12Z

code/modules/surgery/bodypart_manipulation.dm

+						var/mob/living/simple_animal/borer/borer = target.has_brain_worms()
+						if(borer)
+							borer.detatch() //Should remove borer if the brain is removed - RR
+					remove_from_cavity(user, target, choosen_object, BP, tool)


этот прок, хоть и, технически, выполняет то, что хочется (форсмувает на пол и пишет в чат), используется для вещей в BP.implants, а не чего угодно в хумане. волшебным образом работающий костыль

так вещью в BP.implants может оказаться все что угодно, для этого даже есть отдельная категория "Else"

BP.implants - то, что имплантировано в человека - то, что изначально в нём находится не должно

так вещью в BP.implants может оказаться все что угодно, для этого даже есть отдельная категория "Else"

как это вообще отвечает на мое ревью?

я понимаю, что ты хочешь видеть что-то другое вместо remove_from_cavity, но в каких случаях текущий вариант может что-то поломать?

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

4310V343k · 2024-01-07T04:51:03Z

code/modules/surgery/bodypart_manipulation.dm

 	var/obj/item/organ/external/chest/BP = target.get_bodypart(target_zone)
 	user.visible_message("<span class='warning'>[user]'s hand slips, scraping tissue inside [target]'s [BP.name] with \the [tool]!</span>", \
 	"<span class='warning'>Your hand slips, scraping tissue inside [target]'s [BP.name] with \the [tool]!</span>")
 	BP.take_damage(20, 0, DAM_SHARP|DAM_EDGE, tool)
-	if (BP.implants.len)
+	if (BP.implants.len || BP.bodypart_organs.len)


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

имплантом считается любая вещь внутри хумана, не только сами импланты

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

прежде чем что-то сказать я это проверяю

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

прежде чем что-то сказать я это проверяю

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

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

4310V343k

надо было, конечно, в один ревью закинуть, но я чето тупанул

maleyvich · 2024-01-07T07:50:30Z

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

TauKitty · 2024-04-09T11:25:55Z

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/kibernetika/42735/10

maleyvich added 5 commits December 29, 2023 16:09

firststep

082e103

Revert "firststep"

e93e0d7

This reverts commit 082e103.

firststep

4f41785

fix

90d1b4b

Update surgery.dmi

ec4b7e8

TauKitty added the Sprites label Dec 29, 2023

maleyvich marked this pull request as draft December 29, 2023 10:31

TauKitty requested a review from a team December 29, 2023 10:31

maleyvich added 2 commits December 29, 2023 16:51

ipcpumpa

8d13316

Update surgery.dmi

3337c05

maleyvich added 2 commits December 29, 2023 17:20

resprite heart

2925012

Update organ_internal.dm

7e96491

maleyvich added 3 commits December 29, 2023 17:54

human lungs

448fb2a

vox lungs

5f91706

skrell lungs

25d8711

fix

2185855

maleyvich added 2 commits December 30, 2023 11:28

diona lungs

ecda74a

surgery

7fec4ee

KandreyKandrey reviewed Jan 3, 2024

View reviewed changes

code/modules/organs/organ.dm Show resolved Hide resolved

maleyvich added 2 commits January 4, 2024 10:11

organsforroboticsandmed

37b7cd8

Merge branch 'heartsprites' of https://github.com/maleyvich/TauCetiCl…

9e99f6c

…assic into heartsprites

TauKitty requested a review from a team January 4, 2024 04:12

null owner

ea60532

sprites

1235144

NinjaPikachuska reviewed Jan 4, 2024

View reviewed changes

code/modules/surgery/implant.dm Show resolved Hide resolved

NinjaPikachuska reviewed Jan 4, 2024

View reviewed changes

code/modules/surgery/implant.dm Outdated Show resolved Hide resolved

maleyvich added 3 commits January 4, 2024 10:49

rename

7d1a354

embed -= internal

2867e34

Update organ_internal.dm

25af026

NinjaPikachuska reviewed Jan 5, 2024

View reviewed changes

maleyvich and others added 5 commits January 6, 2024 00:20

Update code/modules/mob/living/carbon/brain/MMI.dm

e35d558

Co-authored-by: NinjaPikachuska <89906909+NinjaPikachuska@users.noreply.github.com>

fix

a6bdbe9

oops

166a5d5

Update bodypart_manipulation.dm

d27314f

ohoho

1559c71

4310V343k reviewed Jan 7, 2024

View reviewed changes

4310V343k suggested changes Jan 7, 2024

View reviewed changes

maleyvich closed this Jan 7, 2024

Даём возможность пощупать органы и вырывать их #12653

Даём возможность пощупать органы и вырывать их #12653

Conversation

maleyvich commented Dec 29, 2023 • edited

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

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

Авторство

Чеинжлог

TauKitty commented Dec 29, 2023

Changelog status: ✔️

maleyvich commented Dec 29, 2023

UDaV73rus commented Dec 29, 2023

maleyvich commented Dec 29, 2023

Chip11-n commented Dec 29, 2023

maleyvich commented Dec 29, 2023

KIBORG04 commented Dec 29, 2023

maleyvich commented Dec 29, 2023 • edited

maleyvich commented Dec 29, 2023

Deahaka commented Dec 29, 2023

NinjaPikachuska commented Dec 29, 2023 • edited

maleyvich commented Dec 30, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MrPerenka commented Jan 3, 2024

maleyvich commented Jan 4, 2024

maleyvich commented Jan 4, 2024

NinjaPikachuska left a 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

4310V343k Jan 7, 2024 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

4310V343k Jan 7, 2024 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

4310V343k Jan 7, 2024 • edited

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

4310V343k Jan 7, 2024 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

4310V343k Jan 7, 2024 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

4310V343k Jan 7, 2024 • edited

Choose a reason for hiding this comment

4310V343k left a comment

Choose a reason for hiding this comment

maleyvich commented Jan 7, 2024

TauKitty commented Apr 9, 2024

maleyvich commented Dec 29, 2023 •

edited

maleyvich commented Dec 29, 2023 •

edited

NinjaPikachuska commented Dec 29, 2023 •

edited

4310V343k Jan 7, 2024 •

edited

4310V343k Jan 7, 2024 •

edited

4310V343k Jan 7, 2024 •

edited

4310V343k Jan 7, 2024 •

edited

4310V343k Jan 7, 2024 •

edited

4310V343k Jan 7, 2024 •

edited