AspireMint reports: [Ferns] Woodcutting "Giant fer ... #3738

New Issue

yourland-report · 2023-02-05T17:49:27Z

yourland-report commented

2023-02-05 17:49:27 +00:00

AspireMint reports a bug:

[Ferns] Woodcutting "Giant fern trunk" gives 1 trunk to inventory and 1 is dropped.

Player position:

{
	y = 55.5,
	z = 21488.673828125,
	x = -405.12799072266
}

Player look:

{
	y = 0.51937001943588,
	z = -0.31568887829781,
	x = 0.79410034418106
}

Player information:

{
	max_jitter = 0.59799998998642,
	connection_uptime = 7461,
	serialization_version = 29,
	patch = 0,
	lang_code = "",
	max_rtt = 0.63099998235703,
	version_string = "5.6.0",
	avg_rtt = 0.032999999821186,
	major = 5,
	state = "Active",
	min_rtt = 0.030999999493361,
	ip_version = 6,
	avg_jitter = 0,
	formspec_version = 6,
	protocol_version = 41,
	minor = 6,
	min_jitter = 0
}

Player meta:

{
	fields = {
		repellant = "0",
		["stamina:level"] = "8",
		crafted = "21629",
		yl_church = "return {[\"last_death\"] = {[\"y\"] = 2, [\"z\"] = 24054, [\"x\"] = -50}, [\"last_heal\"] = 1674332798, [\"last_death_portal\"] = 1675612090}",
		inflicted_damage = "5602370",
		["ocean_build.ocean_built"] = "12",
		["3d_armor_inventory"] = "return {\"3d_armor:boots_crystal 1 7260\", \"3d_armor:enchanted_helmet_bronze_strong 1 29040\", \"3d_armor:enchanted_leggings_bronze_strong 1 29040\", \"3d_armor:enchanted_chestplate_bronze_strong 1 29040\", \"shields:shield_bronze 1 29040\", \"\"}",
		["stamina:exhaustion"] = "148",
		yl_commons_player_joined = "1675611991",
		partychat = "party",
		hud_state = "on",
		played_time = "3457681",
		["ocean_build.last_warning"] = "1.65674e+09",
		yl_commons_thankyou = "100",
		punch_count = "258655",
		["signslib:pos"] = "(39,4,22470)",
		arenalib_infobox_arenaID = "0",
		["ocean_build.forbidden"] = "true",
		["stamina:poisoned"] = "no",
		jointime = "1619715757",
		bitten = "0",
		digged_nodes = "656239",
		placed_nodes = "125504",
		died = "164",
		xp = "757015",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"water_life:croc_bag\"}",
		yl_commons_player_created = "1619715757"
	}
}

Log identifier


[MOD] yl_report log identifier = IJLORO0gtGDsmjc9By4Ku5Eejz75W40K

Profiler save:

profile-20230205T174927.json_prettyEE

Status:

# Server: version: 5.6.1-yl | game: Minetest Game | uptime: 1d 6h 11min 1s | max lag: 3.89s | clients (28/52): AliasAlreadyTaken, arual, AspireMint, AzaThoth, Bailiff, Bla, CaramelPlant, Chazz, daydream, Elias, flux, Giuseppe, jackofthebean000, JeCel, Kalek, karlbrock, labrat, Lucien, michi16bit, Murmel, niceride, Parrish, pasqualino, rewired_X, rheo, Service, Sugar0, Sysmatic

Teleport command:

/teleport xyz -405 56 21489

Compass command:

/give_compass Construction IJLORO0gtGDsmjc9By4Ku5Eejz75W40K D2691E -405 56 21489

AspireMint reports a bug: > [Ferns] Woodcutting "Giant fern trunk" gives 1 trunk to inventory and 1 is dropped. Player position: ``` { y = 55.5, z = 21488.673828125, x = -405.12799072266 } ``` Player look: ``` { y = 0.51937001943588, z = -0.31568887829781, x = 0.79410034418106 } ``` Player information: ``` { max_jitter = 0.59799998998642, connection_uptime = 7461, serialization_version = 29, patch = 0, lang_code = "", max_rtt = 0.63099998235703, version_string = "5.6.0", avg_rtt = 0.032999999821186, major = 5, state = "Active", min_rtt = 0.030999999493361, ip_version = 6, avg_jitter = 0, formspec_version = 6, protocol_version = 41, minor = 6, min_jitter = 0 } ``` Player meta: ``` { fields = { repellant = "0", ["stamina:level"] = "8", crafted = "21629", yl_church = "return {[\"last_death\"] = {[\"y\"] = 2, [\"z\"] = 24054, [\"x\"] = -50}, [\"last_heal\"] = 1674332798, [\"last_death_portal\"] = 1675612090}", inflicted_damage = "5602370", ["ocean_build.ocean_built"] = "12", ["3d_armor_inventory"] = "return {\"3d_armor:boots_crystal 1 7260\", \"3d_armor:enchanted_helmet_bronze_strong 1 29040\", \"3d_armor:enchanted_leggings_bronze_strong 1 29040\", \"3d_armor:enchanted_chestplate_bronze_strong 1 29040\", \"shields:shield_bronze 1 29040\", \"\"}", ["stamina:exhaustion"] = "148", yl_commons_player_joined = "1675611991", partychat = "party", hud_state = "on", played_time = "3457681", ["ocean_build.last_warning"] = "1.65674e+09", yl_commons_thankyou = "100", punch_count = "258655", ["signslib:pos"] = "(39,4,22470)", arenalib_infobox_arenaID = "0", ["ocean_build.forbidden"] = "true", ["stamina:poisoned"] = "no", jointime = "1619715757", bitten = "0", digged_nodes = "656239", placed_nodes = "125504", died = "164", xp = "757015", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"water_life:croc_bag\"}", yl_commons_player_created = "1619715757" } } ``` Log identifier ``` [MOD] yl_report log identifier = IJLORO0gtGDsmjc9By4Ku5Eejz75W40K ``` Profiler save: ``` profile-20230205T174927.json_prettyEE ``` Status: ``` # Server: version: 5.6.1-yl | game: Minetest Game | uptime: 1d 6h 11min 1s | max lag: 3.89s | clients (28/52): AliasAlreadyTaken, arual, AspireMint, AzaThoth, Bailiff, Bla, CaramelPlant, Chazz, daydream, Elias, flux, Giuseppe, jackofthebean000, JeCel, Kalek, karlbrock, labrat, Lucien, michi16bit, Murmel, niceride, Parrish, pasqualino, rewired_X, rheo, Service, Sugar0, Sysmatic ``` Teleport command: ``` /teleport xyz -405 56 21489 ``` Compass command: ``` /give_compass Construction IJLORO0gtGDsmjc9By4Ku5Eejz75W40K D2691E -405 56 21489 ```

AliasAlreadyTaken was assigned by yourland-report

2023-02-05 17:49:27 +00:00

flux added the

1. kind/bug

3. source/integration

labels 2023-02-05 20:07:39 +00:00

flux commented

2023-02-05 20:09:24 +00:00

the bug here is that the trunk nodes are duplicated - for both giant tree fern saplings, and "regular" ones.

choppy doesn't even bother w/ the tree ferns, as they've got their own logic for falling down

however, the leaves for the giant tree don't fall via the default mechanics, nor when using woodcutting.

the bug here is that the trunk nodes are duplicated - for both giant tree fern saplings, and "regular" ones. choppy doesn't even bother w/ the tree ferns, as they've got their own logic for falling down however, the leaves for the giant tree don't fall via the default mechanics, nor when using woodcutting.

flux added the

4. step/ready to QA test

label 2023-02-20 01:57:37 +00:00

flux added this to the 1.1.118 milestone 2023-02-20 01:57:40 +00:00

flux referenced this issue

2023-02-20 17:25:03 +00:00

1.1.118 #3401

AliasAlreadyTaken commented

2023-04-13 21:50:02 +00:00

QA

They're still duplicated, even more than double. If I chop a giant tree fern (sapling: ferns:sapling_giant_tree_fern), //count indicated there are 12 trunks. But I get 1 in my inv and 24 (2x12) fall to the ground to a sum of 25.

QA They're still duplicated, even more than double. If I chop a giant tree fern (sapling: ferns:sapling_giant_tree_fern), //count indicated there are 12 trunks. But I get 1 in my inv and 24 (2x12) fall to the ground to a sum of 25.

AliasAlreadyTaken added the

ugh/QA NOK

label 2023-04-13 21:50:07 +00:00

flux commented

2023-04-14 03:35:09 +00:00

QA

They're still duplicated, even more than double. If I chop a giant tree fern (sapling: ferns:sapling_giant_tree_fern), //count indicated there are 12 trunks. But I get 1 in my inv and 24 (2x12) fall to the ground to a sum of 25.

is this using choppy, or woodcutting? hopefully not both?

> QA > > They're still duplicated, even more than double. If I chop a giant tree fern (sapling: ferns:sapling_giant_tree_fern), //count indicated there are 12 trunks. But I get 1 in my inv and 24 (2x12) fall to the ground to a sum of 25. is this using choppy, or woodcutting? hopefully not both?

AliasAlreadyTaken commented

2023-04-14 03:37:20 +00:00

On the testserver, woodcutting is disabled. There's only choppy.

flux commented

2023-04-14 19:57:31 +00:00

oh, i see, i did add the tree ferns to choppy, because their existing mechanic left flying leaves... i've updated choppy to disable the built-in mechanic. d3e1404eac

oh, i see, i did add the tree ferns to choppy, because their existing mechanic left flying leaves... i've updated choppy to disable the built-in mechanic. https://github.com/fluxionary/minetest-choppy/commit/d3e1404eac23fba822f79d0bbf807aecf602cdfc

AliasAlreadyTaken added

ugh/QA OK

and removed

ugh/QA NOK

labels 2023-04-16 04:30:11 +00:00

AliasAlreadyTaken commented

2023-04-16 04:30:28 +00:00

QA

12/12 works!

QA 12/12 works!

flux added

5. result/fixed