niceride reports: 'Replacing nodes of type "pipe ... #4289

Open
opened 2023-04-26 08:31:57 +00:00 by yourland-report · 10 comments

niceride reports a bug:

'Replacing nodes of type "pipeworks:nodebreaker_off" is not allowed on this server. Replacement failed.'

Player position:

{
	x = 20.513000488281,
	y = -124.49900054932,
	z = 27.92200088501
}

Player look:

{
	x = 0.64921933412552,
	y = -0.34202012419701,
	z = 0.67936474084854
}

Player information:

{
	major = 5,
	version_string = "5.6.1",
	formspec_version = 6,
	protocol_version = 41,
	ip_version = 6,
	min_rtt = 0.16699999570847,
	avg_rtt = 0.18099999427795,
	minor = 6,
	max_jitter = 1.5030000209808,
	avg_jitter = 0.010000005364418,
	connection_uptime = 3935,
	serialization_version = 29,
	patch = 1,
	state = "Active",
	lang_code = "",
	min_jitter = 0,
	max_rtt = 1.7519999742508
}

Player meta:

{
	fields = {
		yl_commons_player_joined = "1682493995",
		yl_church = "return {[\"last_death_portal\"] = 1682297423, [\"last_death\"] = {[\"x\"] = -6565, [\"y\"] = 36, [\"z\"] = -6678}}",
		repellant = "0",
		played_time = "12694879",
		digged_nodes = "1926771",
		placed_nodes = "652507",
		died = "145",
		["3d_armor_inventory"] = "return {\"3d_armor:helmet_crystal 1 9480\", \"\", \"3d_armor:chestplate_nether 1 18960\", \"shields:shield_rainbow 1 3792\", \"3d_armor:leggings_crystal 1 9480\", \"3d_armor:boots_crystal 1 9480\"}",
		jointime = "1660920889",
		bitten = "0",
		["petz:old_override_table"] = "return {[\"sneak_glitch\"] = false, [\"new_move\"] = true, [\"speed\"] = 2, [\"gravity\"] = 1, [\"jump\"] = 1.5, [\"sneak\"] = true}",
		["petz:werewolf"] = "0",
		yl_commons_thankyou = "96",
		["ocean_build.last_warning"] = "1.68196e+09",
		["signslib:pos"] = "(1020,48,816)",
		punch_count = "182309",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}",
		inflicted_damage = "3947850",
		hud_state = "on",
		["ocean_build.ocean_built"] = "1",
		["petz:werewolf_vignette_id"] = "19",
		["stamina:level"] = "7",
		arenalib_infobox_arenaID = "0",
		["stamina:poisoned"] = "no",
		xp = "2170655",
		["petz:lycanthropy"] = "0",
		["stamina:exhaustion"] = "85",
		["petz:werewolf_clan_idx"] = "1",
		crafted = "257825",
		yl_commons_player_created = "1660920889"
	}
}

Log identifier


[MOD] yl_report log identifier = oe6RgODkkEjNargB7ruLuQObQcPZoE78

Profiler save:

profile-20230426T083157.json_prettyEE

Status:

# Server: version: 5.6.1-yl | game: Minetest Game | uptime: 9h 26min 49s | max lag: 0.337s | clients (16/52): 9T9, AliasAlreadyTaken, Allan, Bailiff, Balrog_Dude, Boris, daydream, Dex74, Flippster, HorusDamocles, kailesh, Murmel, niceride, rewired_X, Service, Sysmatic

Teleport command:

/teleport xyz 21 -124 28

Compass command:

/give_compass Construction oe6RgODkkEjNargB7ruLuQObQcPZoE78 D2691E 21 -124 28
niceride reports a bug: > 'Replacing nodes of type "pipeworks:nodebreaker_off" is not allowed on this server. Replacement failed.' Player position: ``` { x = 20.513000488281, y = -124.49900054932, z = 27.92200088501 } ``` Player look: ``` { x = 0.64921933412552, y = -0.34202012419701, z = 0.67936474084854 } ``` Player information: ``` { major = 5, version_string = "5.6.1", formspec_version = 6, protocol_version = 41, ip_version = 6, min_rtt = 0.16699999570847, avg_rtt = 0.18099999427795, minor = 6, max_jitter = 1.5030000209808, avg_jitter = 0.010000005364418, connection_uptime = 3935, serialization_version = 29, patch = 1, state = "Active", lang_code = "", min_jitter = 0, max_rtt = 1.7519999742508 } ``` Player meta: ``` { fields = { yl_commons_player_joined = "1682493995", yl_church = "return {[\"last_death_portal\"] = 1682297423, [\"last_death\"] = {[\"x\"] = -6565, [\"y\"] = 36, [\"z\"] = -6678}}", repellant = "0", played_time = "12694879", digged_nodes = "1926771", placed_nodes = "652507", died = "145", ["3d_armor_inventory"] = "return {\"3d_armor:helmet_crystal 1 9480\", \"\", \"3d_armor:chestplate_nether 1 18960\", \"shields:shield_rainbow 1 3792\", \"3d_armor:leggings_crystal 1 9480\", \"3d_armor:boots_crystal 1 9480\"}", jointime = "1660920889", bitten = "0", ["petz:old_override_table"] = "return {[\"sneak_glitch\"] = false, [\"new_move\"] = true, [\"speed\"] = 2, [\"gravity\"] = 1, [\"jump\"] = 1.5, [\"sneak\"] = true}", ["petz:werewolf"] = "0", yl_commons_thankyou = "96", ["ocean_build.last_warning"] = "1.68196e+09", ["signslib:pos"] = "(1020,48,816)", punch_count = "182309", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}", inflicted_damage = "3947850", hud_state = "on", ["ocean_build.ocean_built"] = "1", ["petz:werewolf_vignette_id"] = "19", ["stamina:level"] = "7", arenalib_infobox_arenaID = "0", ["stamina:poisoned"] = "no", xp = "2170655", ["petz:lycanthropy"] = "0", ["stamina:exhaustion"] = "85", ["petz:werewolf_clan_idx"] = "1", crafted = "257825", yl_commons_player_created = "1660920889" } } ``` Log identifier ``` [MOD] yl_report log identifier = oe6RgODkkEjNargB7ruLuQObQcPZoE78 ``` Profiler save: ``` profile-20230426T083157.json_prettyEE ``` Status: ``` # Server: version: 5.6.1-yl | game: Minetest Game | uptime: 9h 26min 49s | max lag: 0.337s | clients (16/52): 9T9, AliasAlreadyTaken, Allan, Bailiff, Balrog_Dude, Boris, daydream, Dex74, Flippster, HorusDamocles, kailesh, Murmel, niceride, rewired_X, Service, Sysmatic ``` Teleport command: ``` /teleport xyz 21 -124 28 ``` Compass command: ``` /give_compass Construction oe6RgODkkEjNargB7ruLuQObQcPZoE78 D2691E 21 -124 28 ```
AliasAlreadyTaken was assigned by yourland-report 2023-04-26 08:31:57 +00:00

When placing i.e. SGG blocks for layout, and patterning off a certain orientation of existing nodebreaker to replace into the layout. Worked in 117, bug in 118.

When placing i.e. SGG blocks for layout, and patterning off a certain orientation of existing nodebreaker to replace into the layout. Worked in 117, bug in 118.

note also broken for "pipeworks:deployer_off"

note also broken for "pipeworks:deployer_off"

probably #1702 (comment)

flux: i've added all of the restricted mesecons devices to the replacer blacklist

probably https://gitea.your-land.de/your-land/bugtracker/issues/1702#issuecomment-43927 > flux: i've added all of the restricted mesecons devices to the replacer blacklist

c.f. #4236

c.f. #4236
AliasAlreadyTaken added the
1. kind/documentation
label 2023-04-26 13:51:20 +00:00
Member

the current behavior is intended, in order to prevent the "infinite pickaxe" of #1702.

in order to allow replacing nodes conditional on the permissions of the user, we'd need an API which the current replacer mod doesn't provide.

on the other hand, my replacer redo won't remove a node if you can't place the replacement.

the current behavior is intended, in order to prevent the "infinite pickaxe" of #1702. in order to allow replacing nodes *conditional on the permissions of the user*, we'd need an API which the current replacer mod doesn't provide. on the other hand, my replacer redo won't remove a node if you can't place the replacement.

So what is this restriction on replacer redo and mesecons items addressing? I don't understand, there's a bug in replacer redo not updating pipeworks? Is it critical enough to disable all this functionality for all players?

So what is this restriction on replacer redo and mesecons items addressing? I don't understand, there's a bug in replacer redo not updating pipeworks? Is it critical enough to disable all this functionality for all players?

Yes, unfortunately. Having a tool that goes around the placement mechanics IS bad and may create unexpected combinations and things may go wrong. We don't want that.

Yes, unfortunately. Having a tool that goes around the placement mechanics IS bad and may create unexpected combinations and things may go wrong. We don't want that.
Member

there's a bug in replacer redo not updating pipeworks?

there's no such bug. there's a bug in the current replacer fork that we're using, which allows "replacing" a node even if you aren't allowed to place the replacement node, which turns your replacer into an infinite-durability instant pickaxe.

i'm trying to sell my own replacer fork as a solution which avoids the issue by trying to follow the builtin API as close as possible.

> there's a bug in replacer redo not updating pipeworks? there's no such bug. there's a bug in the current replacer fork that we're using, which allows "replacing" a node even if you aren't allowed to place the replacement node, which turns your replacer into an infinite-durability instant pickaxe. i'm trying to sell my own replacer fork as a solution which avoids the issue by trying to follow the builtin API as close as possible.

i'm trying to sell my own replacer fork as a solution which avoids the issue by trying to follow the builtin API as close as possible.

How can we test your replacer fork, is it live on the test server?

> i'm trying to sell my own replacer fork as a solution which avoids the issue by trying to follow the builtin API as close as possible. How can we test your replacer fork, is it live on the test server?
Member

How can we test your replacer fork, is it live on the test server?

it's on my own testserver i've set up to test a bunch of new mechanics i'm working on. it's "fluxtest" on the public server list. poke me if you need help getting started. note that pipeworks is not on that server, nor is there any sort of machine restrictions.

> How can we test your replacer fork, is it live on the test server? it's on my own testserver i've set up to test a bunch of new mechanics i'm working on. it's "fluxtest" on the public server list. poke me if you need help getting started. note that pipeworks is *not* on that server, nor is there any sort of machine restrictions.
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#4289
No description provided.