aliwoto reports: Node replacement tool doesn't ... #1602

New Issue

yourland-report · 2022-03-21T11:41:48Z

yourland-report commented

2022-03-21 11:41:48 +00:00

aliwoto reports a bug:

Node replacement tool doesn't let me set the default replacing block to "stone"; when I point at a stone and press "right-click + shift", it says "Node replacement tool set to: default:cobble" (but I'm not poiting at cobblestone at all), this way it's impossible replace blocks with normal stone....

Player position:

{
	y = 18.5,
	x = -4903.5327148438,
	z = -5109.9853515625
}

Player look:

{
	y = -0.30053931474686,
	x = 0.9407165646553,
	z = 0.15725292265415
}

Player information:

{
	min_rtt = 0.089000001549721,
	max_rtt = 1.8420000076294,
	connection_uptime = 11011,
	max_jitter = 1.7480000257492,
	minor = 6,
	major = 5,
	ip_version = 6,
	formspec_version = 5,
	patch = 0,
	protocol_version = 39,
	serialization_version = 28,
	lang_code = "",
	version_string = "5.6.0-dev-4c35bbdf1-dirty",
	avg_rtt = 0.11800000071526,
	state = "Active",
	avg_jitter = 0.020999997854233,
	min_jitter = 0
}

Player meta:

{
	fields = {
		["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 7320\", \"3d_armor:chestplate_rainbow 1 2904\", \"shields:shield_nether 1 14640\", \"3d_armor:leggings_rainbow 1 2928\", \"nether_mobs:dragon_helmet 1 7320\", \"\"}",
		yl_commons_thankyou = "9",
		jointime = "1645546114",
		yl_commons_player_joined = "1647851911",
		["signslib:pos"] = "(846,6,1988)",
		digged_nodes = "156425",
		bitten = "0",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}",
		partychat = "main",
		yl_church = "return {[\"last_death\"] = {[\"y\"] = 9, [\"x\"] = -1280, [\"z\"] = -2756}, [\"last_death_portal\"] = 1647816194, [\"last_heal\"] = 1646098778}",
		["stamina:level"] = "17.575",
		punch_count = "323639",
		arenalib_infobox_arenaID = "0",
		inflicted_damage = "6396064",
		crafted = "6542",
		played_time = "799704",
		xp = "678541",
		placed_nodes = "56859",
		died = "58",
		hud_state = "on",
		repellant = "0",
		yl_commons_player_created = "1645546114"
	}
}

Log identifier


[MOD] yl_report log identifier = jWe0nFdymv3bDYgF7ojXHeg8gNNzVY5i

Profiler save:

profile-20220321T124147.json_pretty

Status:

# Server: version=5.4.1-yl, uptime=72544.7, max_lag=1.83508, clients={Flippster, Aero, Chache, tubbo, bakaxtreme, GOOOOOD, aliwoto, Bailiff, flux}

Teleport command:

/teleport xyz -4904 19 -5110

Compass command:

/give_compass Construction jWe0nFdymv3bDYgF7ojXHeg8gNNzVY5i D2691E -4904 19 -5110

aliwoto reports a bug: > Node replacement tool doesn't let me set the default replacing block to "stone"; when I point at a stone and press "right-click + shift", it says "Node replacement tool set to: default:cobble" (but I'm not poiting at cobblestone at all), this way it's impossible replace blocks with normal stone.... Player position: ``` { y = 18.5, x = -4903.5327148438, z = -5109.9853515625 } ``` Player look: ``` { y = -0.30053931474686, x = 0.9407165646553, z = 0.15725292265415 } ``` Player information: ``` { min_rtt = 0.089000001549721, max_rtt = 1.8420000076294, connection_uptime = 11011, max_jitter = 1.7480000257492, minor = 6, major = 5, ip_version = 6, formspec_version = 5, patch = 0, protocol_version = 39, serialization_version = 28, lang_code = "", version_string = "5.6.0-dev-4c35bbdf1-dirty", avg_rtt = 0.11800000071526, state = "Active", avg_jitter = 0.020999997854233, min_jitter = 0 } ``` Player meta: ``` { fields = { ["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 7320\", \"3d_armor:chestplate_rainbow 1 2904\", \"shields:shield_nether 1 14640\", \"3d_armor:leggings_rainbow 1 2928\", \"nether_mobs:dragon_helmet 1 7320\", \"\"}", yl_commons_thankyou = "9", jointime = "1645546114", yl_commons_player_joined = "1647851911", ["signslib:pos"] = "(846,6,1988)", digged_nodes = "156425", bitten = "0", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}", partychat = "main", yl_church = "return {[\"last_death\"] = {[\"y\"] = 9, [\"x\"] = -1280, [\"z\"] = -2756}, [\"last_death_portal\"] = 1647816194, [\"last_heal\"] = 1646098778}", ["stamina:level"] = "17.575", punch_count = "323639", arenalib_infobox_arenaID = "0", inflicted_damage = "6396064", crafted = "6542", played_time = "799704", xp = "678541", placed_nodes = "56859", died = "58", hud_state = "on", repellant = "0", yl_commons_player_created = "1645546114" } } ``` Log identifier ``` [MOD] yl_report log identifier = jWe0nFdymv3bDYgF7ojXHeg8gNNzVY5i ``` Profiler save: ``` profile-20220321T124147.json_pretty ``` Status: ``` # Server: version=5.4.1-yl, uptime=72544.7, max_lag=1.83508, clients={Flippster, Aero, Chache, tubbo, bakaxtreme, GOOOOOD, aliwoto, Bailiff, flux} ``` Teleport command: ``` /teleport xyz -4904 19 -5110 ``` Compass command: ``` /give_compass Construction jWe0nFdymv3bDYgF7ojXHeg8gNNzVY5i D2691E -4904 19 -5110 ```

AliasAlreadyTaken was assigned by yourland-report

2022-03-21 11:41:48 +00:00

aliwoto commented

2022-03-21 11:44:25 +00:00

I noticed something about this bug
if I have "stone" item in my inventory, node replacement works as intended, there is no issue
but if I don't have stones (nor do I have any cobblestones) in my inventory, it recognizes normal stones as cobblestones

I noticed something about this bug if I have "stone" item in my inventory, node replacement works as intended, there is no issue but if I don't have stones (nor do I have any cobblestones) in my inventory, it recognizes normal stones as cobblestones

Bla commented

2022-03-21 11:52:22 +00:00

Interesting, my guess the replacer notices that you don´t have the matching material and offers what you would get if you digged that node and place the result.

Curious what happens if you try to replace that stone, will it remove the stone and place cobblestone without a change in your inventory?

Interesting, my guess the replacer notices that you don´t have the matching material and offers what you would get if you digged that node and place the result. Curious what happens if you try to replace that stone, will it remove the stone and place cobblestone without a change in your inventory?

flux commented

2022-03-21 15:53:22 +00:00

i believe this is intended behavior, though it is slightly odd.

i believe this is intended behavior, though it *is* slightly odd.

AliasAlreadyTaken commented

2022-03-31 15:41:48 +00:00

Cannot reproduce.

I had cobble and default:stone in my inv, replacer in hand, no creative, no nothing.

I pointed and shift-rightclicked at default:stone. It said

(0 0) default:stone

Then I tried to place the stone from my inventory, I rightclicked and the default:stone was set. I left clicked and the block I pointed to was replaced with the default:stone from my inv, also the pointed to block was brought into my inventory.

Can someone please show me how to repro?

Cannot reproduce. I had cobble and default:stone in my inv, replacer in hand, no creative, no nothing. I pointed and shift-rightclicked at default:stone. It said ``` (0 0) default:stone ``` Then I tried to place the stone from my inventory, I rightclicked and the default:stone was set. I left clicked and the block I pointed to was replaced with the default:stone from my inv, also the pointed to block was brought into my inventory. Can someone please show me how to repro?

AliasAlreadyTaken added the

1. kind/bug

4. step/help wanted

labels 2022-03-31 15:42:02 +00:00

AliasAlreadyTaken commented

2022-03-31 15:47:09 +00:00

Issue appears when you don't have default:stone in your inv.

AliasAlreadyTaken removed the

4. step/help wanted

label 2022-03-31 15:47:16 +00:00

flux added

1. kind/documentation

and removed

1. kind/bug

labels 2022-04-07 21:29:38 +00:00

flux added this to the flux's TODO list project 2022-07-02 20:10:37 +00:00

flux commented

2022-07-02 20:10:56 +00:00

another thing that'd work differently when using my replacer variant https://github.com/fluxionary/minetest-replacer_redo

AliasAlreadyTaken commented

2022-08-08 05:45:19 +00:00

We'll eventually need to come to a technical comparison between the now-updated replacer mod of Sokomine and the rewrite by flux. Currently we're using the swissalps version and there are plenty of others out there. I think we'd do MT a favour if we tried to unify them.

But: https://xkcd.com/927/

We'll eventually need to come to a technical comparison between the now-updated replacer mod of Sokomine and the rewrite by flux. Currently we're using the swissalps version and there are plenty of others out there. I think we'd do MT a favour if we tried to unify them. But: https://xkcd.com/927/

flux commented

2022-08-08 16:32:13 +00:00

We'll eventually need to come to a technical comparison between the now-updated replacer mod of Sokomine and the rewrite by flux. Currently we're using the swissalps version and there are plenty of others out there. I think we'd do MT a favour if we tried to unify them.

But: https://xkcd.com/927/

swissalps has gone in a very different direction than i have, i'm not sure there's a way to unify them. i'm not trying to create a new "standard", but rather to create a tool to solve a specific problem that's somewhat similar to the existing tool. swissalps version tries to do things "low level"; i'm trying to make the most use of the high-level game APIs to achieve better compatibility w/ other mods.

> We'll eventually need to come to a technical comparison between the now-updated replacer mod of Sokomine and the rewrite by flux. Currently we're using the swissalps version and there are plenty of others out there. I think we'd do MT a favour if we tried to unify them. > > But: https://xkcd.com/927/ swissalps has gone in a very different direction than i have, i'm not sure there's a way to unify them. i'm not trying to create a new "standard", but rather to create a tool to solve a specific problem that's somewhat similar to the existing tool. swissalps version tries to do things "low level"; i'm trying to make the most use of the high-level game APIs to achieve better compatibility w/ other mods.

flux added the

4. step/discussion

label 2022-10-26 17:43:52 +00:00

flux referenced this issue

2022-11-17 18:48:57 +00:00

daydream reports: I used replacer to preserve th ... #3067

Sign in to join this conversation.