Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

more uncraftable items #41

Open
7 of 25 tasks
Tracked by #128
wsor4035 opened this issue Aug 6, 2022 · 22 comments
Open
7 of 25 tasks
Tracked by #128

more uncraftable items #41

wsor4035 opened this issue Aug 6, 2022 · 22 comments
Labels
bug Something isn't working

Comments

@wsor4035
Copy link
Collaborator

wsor4035 commented Aug 6, 2022

  • homedecor:glowlight_quarter_14
  • homedecor:wood_table_large_square
  • homedecor:glass_table_large_square
  • homedecor:wall_lamp_on
  • homedecor:bed_extended
  • homedecor:toilet_open
  • homedecor:kitchen_cabinet_colorable_with_drawers_steel
  • homedecor:kitchen_cabinet_colorable_with_drawers_granite
  • homedecor:kitchen_cabinet_colorable_with_drawers
  • homedecor:desk_lamp_14
  • homedecor:ceiling_lamp_14
  • homedecor:glowlight_half_14
  • homedecor:standing_lamp_14
  • homedecor:plasma_ball_on
  • homedecor:hanging_lantern_14
  • homedecor:ceiling_lantern_14
  • homedecor:table_lamp_14
  • homedecor:ground_lantern_14
  • homedecor:rope_light_on_ceiling_on
  • homedecor:glowlight_small_cube_14
  • homedecor:plasma_lamp_14
  • homedecor:lattice_lantern_large_14
  • homedecor:rope_light_on_floor_on
  • homedecor:kitchen_cabinet_colorable_with_drawers_marble
  • homedecor:lattice_lantern_small_14

total items: 673
ignored items: 281
missing items: 25

https://xkcd.com/1205/

generated by:

minetest.register_chatcommand("hd_test", {
	description = "does stuff, if you cant figure it out, dont use it",
	func = function()
		local valid = {
			homedecor = true,
			building_blocks = true,
			fake_fire = true,
			itemframes = true,
			lavalamp = true,
		}

		local output = {}
		local hd_count, ig_count, missing_count = 0, 0, 0
		for k, v in pairs(minetest.registered_items) do
			if valid[k:split(":")[1]] then
				hd_count = hd_count + 1
				local crafts = minetest.get_all_craft_recipes(k)
				if not crafts then
					if v.groups and v.groups.not_in_creative_inventory == 1 then
						ig_count = ig_count + 1
					else
						missing_count = missing_count + 1
						table.insert(output, "* [ ] " .. k)
					end
				end
			end
		end
		table.insert(output, "total items: " .. hd_count)
		table.insert(output, "ignored items: " .. ig_count)
		table.insert(output, "missing items: " .. missing_count)
		minetest.chat_send_all(dump(output))
		minetest.log("error", dump(output))

		--minetest.get_all_craft_recipes(query item)
	end
})

it may be worth improving this to recursively look through crafts results to validate all node names as well, but see the xkcd, something i slapped together in a few minutes

@BuckarooBanzay
Copy link
Member

your snippet is pretty neat, any objections if i use that somewhere else? 😏

@S-S-X
Copy link
Member

S-S-X commented Aug 6, 2022

But keep in mind that xkcd does not apply to hobbies so go on and keep working on it 😄

@SwissalpS

This comment was marked as outdated.

@BuckarooBanzay
Copy link
Member

some of these don't need a recipe. e.g. toilet_open -> craft toilet then click on it to open.

Then it should fall into the not_in_creative_inventory category i think..

@Niklp09 Niklp09 added the bug Something isn't working label Aug 18, 2022
@dacmot
Copy link

dacmot commented Aug 31, 2022

Note that this check can be made with qa_block. Also it only reports the *_14 items as being uncraftable.

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

toliet is craftable, it uses groups, probably why it wasnt found

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

image
image
glow light 14 is craftable via this way, side note, the glowlight code is rather cursed to read

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

homedecor:wood_table_large_square
homedecor:glass_table_large_square
dont seem to exist?

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

image
wall lamp

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

bed extended fixed a598fa3

image

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

kitchen cabinets are covered by #4 (comment)

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

image
desk lamp

@wsor4035
Copy link
Collaborator Author

wsor4035 commented Sep 2, 2022

tldr it seems most of these are rather cursed crafting paths

@ulamthelucky

This comment was marked as off-topic.

@wsor4035

This comment was marked as off-topic.

taylorskalyo added a commit to taylorskalyo/homedecor_modpack that referenced this issue May 7, 2023
taylorskalyo added a commit to taylorskalyo/homedecor_modpack that referenced this issue May 7, 2023
taylorskalyo added a commit to taylorskalyo/homedecor_modpack that referenced this issue May 7, 2023
taylorskalyo added a commit to taylorskalyo/homedecor_modpack that referenced this issue May 7, 2023
wsor4035 pushed a commit that referenced this issue May 7, 2023
* fix large square table top recipes (#41)

* fix cabinet with drawers recipes (#41)
@Lazerbeak12345
Copy link

Lazerbeak12345 commented Mar 28, 2024

Craft conflict - partially noted, but now I get the other option between these two (IE OP is now backwards, but still a well known problem.)

My idea is we could do the vertical doors -> drawers -> half thingy instead. (This idea is mostly why I'm bothering to comment).

image
image

PS: yes that idea would lead to a very cursed craft order, but at least it isn't conflicting... Alternative might be to copy a slab recipe to get the half one.

@monk-afk

This comment was marked as resolved.

@Niklp09

This comment was marked as resolved.

@wsor4035

This comment was marked as resolved.

@wsor4035

This comment was marked as resolved.

@wsor4035

This comment was marked as resolved.

@monk-afk
Copy link
Contributor

monk-afk commented May 20, 2024

If the aliases are registered before the crafts registrations, the craft recipes work fine. Is this still a mtg issue?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants