Fabricator recycling #12191

AirBlack · 2023-09-28T14:59:09Z

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

Добавляет переработку фабрикаторами изученных предметов. Разбирает на ресурсы, требуемые для производства такого же предмета, но умноженное на 0.8. В зависимости от размера предмета разное время разбора.

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

Темы обсуждения:

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

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

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

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

Ну и шахтеры могут разбирать всякое говно по типу ригов.

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

Авторство

AirBlack - код и идея

Чеинжлог

🆑 AirBlack

rscadd: Возможность перерабатывать предметы в фабрикаторе.

TauKitty · 2023-09-28T14:59:12Z

Changelog status: ✔️

…ator with item if fabricator cannot disassemble it

Fenriros · 2023-10-02T06:05:05Z

Я тебе делаю глубокий поклон за это.

code/game/mecha/mech_fabricator.dm

…rRecycling

AirBlack · 2023-10-10T12:16:16Z

а че не так то

volas · 2023-10-11T06:47:23Z

могу вернуть в тм, мне просто показалось оно еще ворк ин прогресс

AirBlack · 2023-10-13T19:11:08Z

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

volas · 2023-10-26T02:10:07Z

do it

AirBlack · 2023-10-30T17:07:43Z

а я забыл совсем ща поковыряюсь

…rRecycling

AirBlack · 2023-10-30T17:45:25Z

на do_after переделал, ща добавлю исключения по contents и нашел баг с рецуклингом мехов, то что считается только chasis по ресурсам

…ator is recycling, added blocked types, fixed cable layer runtime on destroy

volas · 2023-12-08T06:15:40Z

Я пока думаю. Хранить такие листы текстом действительно может иметь смысл: initial полезный, и это еще более эффективно с точки зрения памяти. Особенно если обращения к нему редкие и оверхед с чтением такого листа незначительный.

Еще подобным образом и техуровни сделаны, можно сделать стандартом. Попробовать написать красивые макросы для инициализации, может быть для удобства сохранять лист как json (конвертация будет просто через нативные jsonencode/jsondecode для рантайм доступа и редактирования).

Я чуть позже еще вернусь к этому.

AirBlack · 2023-12-08T09:34:04Z

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

AirBlack · 2023-12-11T12:50:49Z

зацените мой диагноз

F32.10 – Депрессивный эпизод средней степени без соматических симптомов

AndroBetel · 2023-12-11T13:32:23Z

педовка ты

AirBlack · 2023-12-12T07:44:11Z

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

volas · 2023-12-14T08:14:13Z

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

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

construction = MAT_METAL+"=200;"+MAT_GLASS+"=50"

на

construction = "[MAT_METAL]=200;[MAT_GLASS]=50"

AirBlack · 2023-12-14T15:04:15Z

Я сначала так попробовал, но интерполяция строк через [penis] не работает в статике даже если внутри статические константы. Ошибка компиляции, ругается на то что динамическое чето там. Только через плюс

volas · 2023-12-15T03:09:01Z

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

AirBlack · 2023-12-17T17:24:37Z

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

volas

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

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

volas · 2023-12-19T07:06:39Z

code/game/machinery/doors/airlock.dm

@@ -569,7 +569,7 @@ var/global/list/airlock_overlays = list()
 	if (isElectrified())
 		if (isitem(mover))
 			var/obj/item/i = mover
-			if (i.m_amt)
+			if (params2numberlist(i.construction)[MAT_METAL])


это ппц такое делать в CanPass, сделай лучше флаг дверям conductive = FALSE и в atom_init определяй как TRUE при этом условии. Тут останется флаг проверять.

volas · 2023-12-19T07:08:03Z

code/__HELPERS/_lists.dm

+	var/temp_list = list()
+	var/params_list = params2list(params)
+
+	for(var/key as anything in params_list)


Suggested change

for(var/key as anything in params_list)

for(var/key in params_list)

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

volas · 2023-12-19T07:10:54Z

code/__HELPERS/_lists.dm

+/proc/numberlist2params(list/number_list)
+	var/temp_list = list()
+
+	for(var/key as anything in number_list)


Suggested change

for(var/key as anything in number_list)

for(var/key in number_list)

volas · 2023-12-19T07:19:55Z

code/game/objects/objs.dm

@@ -1,7 +1,6 @@
 /obj
 	//var/datum/module/mod		//not used
-	var/m_amt = 0	// metal
-	var/g_amt = 0	// glass
+	var/construction = "" // Resources used for construction


Я хотел предложить переименовать в materials, но забавный факт - кто-то умудрился уже объявить эту переменную у /obj/item, и кто-то даже умудрился скопипастить массивы материалов с тг в паре мест, которые естественно нигде не используются и ни на что не влияют.

Потому я бы сначала предложил текущие материалы, типо

TauCetiClassic/code/game/objects/items/weapons/tools.dm

Line 573 in 5d2b768

materials = list(MAT_METAL=150, MAT_SILVER=50)

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

И уже после спокойно переименовать var/construction в более понятный и подходящий тут (на мой взгляд) var/materials

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

volas · 2023-12-19T07:45:42Z

code/game/objects/objs.dm

+/obj/proc/can_be_disassembled()
+	/*
+	Returns TRUE if this item can be safely disassembled by recycling.
+	TODO: Add more implementations and use this proc in RnD deconstruction and autolathe recycling
+	*/
+	return contents.len == 0


Я бы предпочел, чтоб автолаты и материалы были в разных ПР-ах, так было бы удобнее обсуждать и мержить. Но раз уж закоммитил всё вместе уже, то ладно.

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

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

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

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

volas · 2023-12-19T08:08:05Z

code/game/objects/items/devices/flashlight.dm

@@ -263,8 +261,7 @@
 	icon_state = "floor" //not a slime extract sprite but... something close enough!
 	item_state = "slime"
 	w_class = SIZE_MINUSCULE
-	m_amt = 0
-	g_amt = 0
+	construction = MAT_METAL+"=0;"+MAT_GLASS+"=0"


Suggested change

construction = MAT_METAL+"=0;"+MAT_GLASS+"=0"

construction = null

volas · 2023-12-19T08:08:19Z

code/game/objects/items/devices/radio/headset.dm

@@ -3,8 +3,7 @@
 	desc = "An updated, modular intercom that fits over the head. Takes encryption keys."
 	icon_state = "headset"
 	item_state = "headset"
-	g_amt = 0
-	m_amt = 75
+	construction = MAT_METAL+"=75;"+MAT_GLASS+"=0"


Suggested change

construction = MAT_METAL+"=75;"+MAT_GLASS+"=0"

construction = MAT_METAL+"=75"

volas · 2023-12-19T08:09:02Z

code/game/objects/items/stacks/sheets/glass.dm

@@ -25,7 +25,7 @@
 	desc = "HOLY SHEET! That is a lot of glass."
 	singular_name = "glass sheet"
 	icon_state = "sheet-glass"
-	g_amt = 0
+	construction = MAT_GLASS+"=0"


Suggested change

construction = MAT_GLASS+"=0"

construction = null

volas · 2023-12-19T08:09:14Z

code/game/objects/items/stacks/sheets/glass.dm

@@ -177,8 +176,7 @@
 	desc = "Glass which seems to have rods or something stuck in them."
 	singular_name = "reinforced glass sheet"
 	icon_state = "sheet-rglass"
-	g_amt = 0
-	m_amt = 0
+	construction = MAT_METAL+"=0;"+MAT_GLASS+"=0"


Suggested change

construction = MAT_METAL+"=0;"+MAT_GLASS+"=0"

construction = null

volas · 2023-12-19T08:09:29Z

code/game/objects/items/stacks/sheets/sheet_types.dm

@@ -107,7 +107,7 @@ var/global/list/datum/stack_recipe/metal_recipes = list (
 	desc = "Sheets made out off metal. It has been dubbed Metal Sheets."
 	singular_name = "metal sheet"
 	icon_state = "sheet-metal"
-	m_amt = 0
+	construction = MAT_METAL+"=0"


Suggested change

construction = MAT_METAL+"=0"

construction = null

github-actions · 2024-01-04T04:18:55Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

github-actions · 2024-01-11T04:35:00Z

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

AirBlack · 2024-01-19T01:59:45Z

@TauCetiStation/maintainers переоткройте я на выхах поковыряю

LudwigVonChesterfield · 2024-01-23T23:55:20Z

code/defines/obj/hydro.dm

@@ -1429,7 +1428,7 @@
 	force = 5.0
 	throwforce = 7.0
 	w_class = SIZE_TINY
-	m_amt = 2550
+	construction = MAT_METAL+"=2550"


@volas Точно не лучше было бы иметь отдельный список var/list/construction_materials и var/list/materials которые сейчас есть в штучке?

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

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

в 515 через :: можно получать листы по типу объекта

докажи

:: просто шорткат для init, он не поможет. Списки инициализируются вместе с объектом, на своём скрытом шаге инициализации (типо New), и до инициализации объекта не доступны.

Хорошие новости - Люммокс согласен, что это проблема, и рассматривает варианты как это можно решить https://www.byond.com/forum/post/2904278#comment26636169

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

Но нужно будет еще замерить встроенным профайлером память до и после чисто из интереса.

тогда ПР ждет 515? если да отметьте там чтоб пр не закрывался там хз

мы уже на 515, делай

If A::B isn't a static var, then it's equivalent to initial(A:B). If A is a constant type path, the compiler will go even
further by compiling this expression as the actual initial value instead of doing a runtime lookup.

оказывается, это не работает так, как нужно 😔

skill issue

LudwigVonChesterfield · 2024-01-30T19:35:24Z

@AirBlack Закрепил ПР, в ожидании 515.

Я бы предложил заранее выкатить ПР с списком, теперь когда всё будет на списках думаю могу взять ревью на себя.

LudwigVonChesterfield · 2024-01-31T20:27:08Z

меня обманули со списками.

AirBlack · 2024-02-01T05:33:49Z

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

AirBlack · 2024-02-05T08:00:35Z

немного занят был на днях простите

github-actions · 2024-02-21T16:47:54Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

AirBlack · 2024-02-21T20:34:07Z

меня разбанили друзья

AirBlack · 2024-02-28T09:14:06Z

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

Mech fabricator recycling

542c8a9

TauKitty added the Feature Новая фича label Sep 28, 2023

AirBlack changed the title ~~Mech fabricator recycling~~ Fabricator recycling Sep 28, 2023

This comment was marked as abuse.

Sign in to view

Removed duplicated code from mining fabricator, added punching fabric…

c063ab7

…ator with item if fabricator cannot disassemble it

This comment was marked as abuse.

Sign in to view

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

This comment was marked as abuse.

Sign in to view

volas reviewed Oct 2, 2023

View reviewed changes

code/game/mecha/mech_fabricator.dm Outdated Show resolved Hide resolved

Merge remote-tracking branch 'upstream/master' into feature/fabricato…

ac0d530

…rRecycling

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

This comment was marked as abuse.

Sign in to view

AirBlack added 2 commits October 30, 2023 23:09

Merge remote-tracking branch 'upstream/master' into feature/fabricato…

b65cafb

…rRecycling

Use do_after instead of use_tool

16daccf

AirBlack added 2 commits October 31, 2023 00:53

Added more advanced check for contents

88282c3

Extracted getting recycled materials proc, blocking actions if fabric…

bd49a35

…ator is recycling, added blocked types, fixed cable layer runtime on destroy

volas requested changes Dec 19, 2023

View reviewed changes

github-actions bot added the Stalled PR label Jan 4, 2024

github-actions bot closed this Jan 11, 2024

Chip11-n reopened this Jan 23, 2024

github-actions bot removed the Stalled PR label Jan 23, 2024

LudwigVonChesterfield reviewed Jan 23, 2024

View reviewed changes

LudwigVonChesterfield added the Pinned ПРы с этим лэйблом будут игнорироваться Stale-ботом label Jan 30, 2024

LudwigVonChesterfield removed the Pinned ПРы с этим лэйблом будут игнорироваться Stale-ботом label Jan 31, 2024

github-actions bot added the Stalled PR label Feb 21, 2024

github-actions bot removed the Stalled PR label Feb 22, 2024

AirBlack closed this Feb 28, 2024

	for(var/key as anything in params_list)
	for(var/key in params_list)

	for(var/key as anything in number_list)
	for(var/key in number_list)

	construction = MAT_METAL+"=0;"+MAT_GLASS+"=0"
	construction = null

	construction = MAT_METAL+"=75;"+MAT_GLASS+"=0"
	construction = MAT_METAL+"=75"

Fabricator recycling #12191

Fabricator recycling #12191

Conversation

AirBlack commented Sep 28, 2023 • edited Loading

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

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

Авторство

Чеинжлог

TauKitty commented Sep 28, 2023

Changelog status: ✔️

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

Fenriros commented Oct 2, 2023

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

AirBlack commented Oct 10, 2023

volas commented Oct 11, 2023

AirBlack commented Oct 13, 2023

volas commented Oct 26, 2023

AirBlack commented Oct 30, 2023

AirBlack commented Oct 30, 2023

volas commented Dec 8, 2023

AirBlack commented Dec 8, 2023

AirBlack commented Dec 11, 2023

AndroBetel commented Dec 11, 2023

AirBlack commented Dec 12, 2023

volas commented Dec 14, 2023

AirBlack commented Dec 14, 2023

volas commented Dec 15, 2023 • edited Loading

AirBlack commented Dec 17, 2023

volas 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

github-actions bot commented Jan 4, 2024

github-actions bot commented Jan 11, 2024

AirBlack commented Jan 19, 2024

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

NinjaPikachuska Jan 31, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LudwigVonChesterfield commented Jan 30, 2024

LudwigVonChesterfield commented Jan 31, 2024

AirBlack commented Feb 1, 2024

AirBlack commented Feb 5, 2024

github-actions bot commented Feb 21, 2024

AirBlack commented Feb 21, 2024

AirBlack commented Feb 28, 2024

AirBlack commented Sep 28, 2023 •

edited

Loading

volas commented Dec 15, 2023 •

edited

Loading

NinjaPikachuska Jan 31, 2024 •

edited

Loading