Azelf reports: Discussion on the max limit of ... #5205

New Issue

yourland-report · 2023-08-26T01:46:11Z

yourland-report commented

2023-08-26 01:46:11 +00:00

Azelf reports a bug:

Discussion on the max limit of the entity detector pending the changes in #4988

Player position:

{
	z = 514.52600097656,
	x = 1321.0980224609,
	y = 63.5
}

Player look:

{
	z = 0.91089588403702,
	x = -0.40517562627792,
	y = -0.078111104667187
}

Player information:

{
	avg_rtt = 0.47600001096725,
	min_jitter = 0,
	max_jitter = 30.542999267578,
	avg_jitter = 0.30300003290176,
	connection_uptime = 7755,
	max_rtt = 30.716999053955,
	patch = 0,
	version_string = "5.7.0",
	state = "Active",
	minor = 7,
	serialization_version = 29,
	major = 5,
	lang_code = "",
	protocol_version = 42,
	formspec_version = 6,
	ip_version = 6,
	min_rtt = 0.17200000584126
}

Player meta:

{
	fields = {
		["stamina:level"] = "12",
		["stamina:poisoned"] = "no",
		hud_state = "on",
		["stamina:exhaustion"] = "74",
		yl_commons_thankyou = "83",
		jointime = "1659363683",
		bitten = "0",
		yl_commons_player_created = "1659363683",
		arenalib_infobox_arenaID = "0",
		xp = "986942",
		inflicted_damage = "4985004",
		["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 4360\", \"3d_armor:chestplate_rainbow 1 1744\", \"shields:shield_rainbow 1 1744\", \"3d_armor:helmet_rainbow 1 1744\", \"3d_armor:leggings_rainbow 1 1744\", \"\"}",
		partychat = "party",
		["ocean_build.ocean_built"] = "11",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}",
		played_time = "5167172",
		["ocean_build.last_warning"] = "1.67602e+09",
		placed_nodes = "246282",
		yl_church = "return {[\"last_heal\"] = 1678197363, [\"last_death\"] = {[\"z\"] = 4622, [\"x\"] = -912, [\"y\"] = 143}, [\"last_death_portal\"] = 1692972695}",
		repellant = "0",
		died = "111",
		digged_nodes = "731305",
		yl_commons_player_joined = "1693006680",
		crafted = "474296",
		punch_count = "234863",
		["signslib:pos"] = "(3475,42,831)"
	}
}

Log identifier


[MOD] yl_report log identifier = eZ3UG3rmz56UwrP2WrtEdH2QBrejmvJV

Profiler save:

profile-20230826T014610.json_prettyEE

Status:

# Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 5h 16min 17s | max lag: 0.392s | clients (16/52): AliasAlreadyTaken, Azelf, BomBuela__bunny279, flux, j0e_it, jackofthebean000, leandro124, MicaelStarfire, nazthelizard122, Parrish, Penelopee, rancid, Service, shanish2, Taiel, yowyowming

Teleport command:

/teleport xyz 1321 64 515

Compass command:

/give_compass Construction eZ3UG3rmz56UwrP2WrtEdH2QBrejmvJV D2691E 1321 64 515

Azelf reports a bug: > Discussion on the max limit of the entity detector pending the changes in #4988 Player position: ``` { z = 514.52600097656, x = 1321.0980224609, y = 63.5 } ``` Player look: ``` { z = 0.91089588403702, x = -0.40517562627792, y = -0.078111104667187 } ``` Player information: ``` { avg_rtt = 0.47600001096725, min_jitter = 0, max_jitter = 30.542999267578, avg_jitter = 0.30300003290176, connection_uptime = 7755, max_rtt = 30.716999053955, patch = 0, version_string = "5.7.0", state = "Active", minor = 7, serialization_version = 29, major = 5, lang_code = "", protocol_version = 42, formspec_version = 6, ip_version = 6, min_rtt = 0.17200000584126 } ``` Player meta: ``` { fields = { ["stamina:level"] = "12", ["stamina:poisoned"] = "no", hud_state = "on", ["stamina:exhaustion"] = "74", yl_commons_thankyou = "83", jointime = "1659363683", bitten = "0", yl_commons_player_created = "1659363683", arenalib_infobox_arenaID = "0", xp = "986942", inflicted_damage = "4985004", ["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 4360\", \"3d_armor:chestplate_rainbow 1 1744\", \"shields:shield_rainbow 1 1744\", \"3d_armor:helmet_rainbow 1 1744\", \"3d_armor:leggings_rainbow 1 1744\", \"\"}", partychat = "party", ["ocean_build.ocean_built"] = "11", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}", played_time = "5167172", ["ocean_build.last_warning"] = "1.67602e+09", placed_nodes = "246282", yl_church = "return {[\"last_heal\"] = 1678197363, [\"last_death\"] = {[\"z\"] = 4622, [\"x\"] = -912, [\"y\"] = 143}, [\"last_death_portal\"] = 1692972695}", repellant = "0", died = "111", digged_nodes = "731305", yl_commons_player_joined = "1693006680", crafted = "474296", punch_count = "234863", ["signslib:pos"] = "(3475,42,831)" } } ``` Log identifier ``` [MOD] yl_report log identifier = eZ3UG3rmz56UwrP2WrtEdH2QBrejmvJV ``` Profiler save: ``` profile-20230826T014610.json_prettyEE ``` Status: ``` # Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 5h 16min 17s | max lag: 0.392s | clients (16/52): AliasAlreadyTaken, Azelf, BomBuela__bunny279, flux, j0e_it, jackofthebean000, leandro124, MicaelStarfire, nazthelizard122, Parrish, Penelopee, rancid, Service, shanish2, Taiel, yowyowming ``` Teleport command: ``` /teleport xyz 1321 64 515 ``` Compass command: ``` /give_compass Construction eZ3UG3rmz56UwrP2WrtEdH2QBrejmvJV D2691E 1321 64 515 ```

AliasAlreadyTaken was assigned by yourland-report

2023-08-26 01:46:11 +00:00

flux added the

1. kind/enhancement

1. kind/balancing

labels 2023-08-26 05:36:28 +00:00

flux commented

2023-08-26 05:41:50 +00:00

azelf felt that having a radius of 64 nodes would be better for creating automated defense systems. i don't disagree. we can change the value easily via moremesecons_entity_detector.max_radius setting. minetest.get_objects_inside_radius needs to be made more efficient, but that's no reason to not extend the range a bit if it's actually useful. the current cost of that call is something like (#all_objects_on_server + 10 * #objects_inside_radius). for most calls, the total number of active objects on the server totally dominates the cost.

azelf felt that having a radius of 64 nodes would be better for creating automated defense systems. i don't disagree. we can change the value easily via `moremesecons_entity_detector.max_radius` setting. `minetest.get_objects_inside_radius` needs to be made more efficient, but that's no reason to not extend the range a bit if it's actually useful. the current cost of that call is something like (#all_objects_on_server + 10 * #objects_inside_radius). for most calls, the total number of active objects on the server totally dominates the cost.

AliasAlreadyTaken commented

2023-08-26 12:59:43 +00:00

I'd rather align that to the max range of existing, similar block like the player detector or the vacuum tube.

Among other things, I do not want long range radar towers detecting Voice attacks.

That's why I'd like to limit the range between 4 and 8.

I'd rather align that to the max range of existing, similar block like the player detector or the vacuum tube. Among other things, I do not want long range radar towers detecting Voice attacks. That's why I'd like to limit the range between 4 and 8.

Bla commented

2023-08-26 13:49:27 +00:00

That's why I'd like to limit the range between 4 and 8.

4 is way to short, players will just place more of them.
8 might work but could be problematic for bigger things like automatic draw bridges.
I think 16 is a nice distance and also easy to visualize for players with displaying block bounds.

> That's why I'd like to limit the range between 4 and 8. 4 is way to short, players will just place more of them. 8 might work but could be problematic for bigger things like automatic draw bridges. I think 16 is a nice distance and also easy to visualize for players with displaying block bounds.

DragonWrangler commented

2023-08-26 16:53:58 +00:00

I agree with Bla

Sokomine commented

2023-08-26 18:18:55 +00:00

Perhaps long range detectors of Voice attacks could be helpful. Right now there has to be someone there in order for an attack to be successful (if the city isn't defended, Voices' troops would march in and despawn after conquering the city..). Those detectors don't have to be physical entities. Might just be a mob announcing that he saw something he/she/it was afraid of. Something like the reports from battles Kika sometimes delivers to us.

👍 2

AliasAlreadyTaken commented

2023-08-27 03:30:13 +00:00

When a city employs sentries, they will announce appoaraching armies or whatever they think is noteworthy. That's a mecahnic of its own, part of the cities update

👍 1

DragonWrangler commented

2023-08-27 13:05:44 +00:00

So basically they will act as the detectors for cities ?

Sign in to join this conversation.