flux reports: keep your machines simple. lim ... #3093

Open
opened 2022-11-21 04:58:09 +00:00 by yourland-report · 8 comments

flux reports a bug:

keep your machines simple. limit the amount of computation that can be done in a mapblock. a single autocrafter should be able to execute once per server step; but add more autocrafters at peril

Player position:

{
	y = 34.5,
	z = 507.79400634766,
	x = 1278.4370117188
}

Player look:

{
	y = -0.99577289819717,
	z = -0.0077496832236648,
	x = 0.091521710157394
}

Player information:

{
	avg_jitter = 0.0090000033378601,
	connection_uptime = 48152,
	max_rtt = 6.3619999885559,
	patch = 1,
	ip_version = 6,
	min_rtt = 0.15600000321865,
	avg_rtt = 0.16599999368191,
	min_jitter = 0,
	max_jitter = 5.2199997901917,
	formspec_version = 6,
	state = "Active",
	serialization_version = 29,
	protocol_version = 41,
	major = 5,
	minor = 6,
	version_string = "5.6.1",
	lang_code = ""
}

Player meta:

{
	fields = {
		bitten = "0",
		["stamina:exhaustion"] = "131",
		played_time = "27416805",
		["petz:werewolf"] = "0",
		placed_nodes = "364762",
		["petz:lycanthropy"] = "1",
		repellant = "0",
		["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}",
		arenalib_infobox_arenaID = "0",
		punch_count = "117182",
		["petz:werewolf_vignette_id"] = "20",
		inflicted_damage = "2058002",
		yl_commons_thankyou = "181",
		["3d_armor_inventory"] = "return {\"3d_armor:chestplate_nether 1 1720\", \"nether_mobs:dragon_boots 1 860\", \"3d_armor:leggings_nether 1 1720\", \"nether_mobs:dragon_shield 1 860\", \"3d_armor:helmet_nether 1 860\", \"\"}",
		yl_commons_player_joined = "1668958568",
		["stamina:poisoned"] = "no",
		xp = "1363043",
		digged_nodes = "1148278",
		["signslib:pos"] = "(770,6,1995)",
		yl_commons_player_created = "1615351173",
		["petz:werewolf_clan_idx"] = "1",
		yl_church = "return {[\"last_heal\"] = 1615504208, [\"last_death_portal\"] = 1668228099, [\"last_death\"] = {[\"x\"] = 6554, [\"y\"] = 2, [\"z\"] = 10487}}",
		["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}",
		crafted = "47767",
		hud_state = "on",
		died = "73",
		["stamina:level"] = "5",
		partychat = "party",
		jointime = "1615351173"
	}
}

Log identifier


[MOD] yl_report log identifier = WjOSwYMsv8ld0ipiNwjwAgkkPFfqPL4W

Profiler save:

profile-20221121T045809.json_prettyEE

Status:

# Server: version: 5.6.1-yl | game: Minetest Game | uptime: 18h 27min 48s | max lag: 1.63s | clients (12/42): AliasAlreadyTaken, Bailiff, bodshfa, daydream, flux, madonna, niceride, Queenfire234312, Service, set, shanish, shanish2

Teleport command:

/teleport xyz 1278 35 508

Compass command:

/give_compass Construction WjOSwYMsv8ld0ipiNwjwAgkkPFfqPL4W D2691E 1278 35 508
flux reports a bug: > keep your machines simple. limit the amount of computation that can be done in a mapblock. a single autocrafter should be able to execute once per server step; but add more autocrafters at peril Player position: ``` { y = 34.5, z = 507.79400634766, x = 1278.4370117188 } ``` Player look: ``` { y = -0.99577289819717, z = -0.0077496832236648, x = 0.091521710157394 } ``` Player information: ``` { avg_jitter = 0.0090000033378601, connection_uptime = 48152, max_rtt = 6.3619999885559, patch = 1, ip_version = 6, min_rtt = 0.15600000321865, avg_rtt = 0.16599999368191, min_jitter = 0, max_jitter = 5.2199997901917, formspec_version = 6, state = "Active", serialization_version = 29, protocol_version = 41, major = 5, minor = 6, version_string = "5.6.1", lang_code = "" } ``` Player meta: ``` { fields = { bitten = "0", ["stamina:exhaustion"] = "131", played_time = "27416805", ["petz:werewolf"] = "0", placed_nodes = "364762", ["petz:lycanthropy"] = "1", repellant = "0", ["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}", arenalib_infobox_arenaID = "0", punch_count = "117182", ["petz:werewolf_vignette_id"] = "20", inflicted_damage = "2058002", yl_commons_thankyou = "181", ["3d_armor_inventory"] = "return {\"3d_armor:chestplate_nether 1 1720\", \"nether_mobs:dragon_boots 1 860\", \"3d_armor:leggings_nether 1 1720\", \"nether_mobs:dragon_shield 1 860\", \"3d_armor:helmet_nether 1 860\", \"\"}", yl_commons_player_joined = "1668958568", ["stamina:poisoned"] = "no", xp = "1363043", digged_nodes = "1148278", ["signslib:pos"] = "(770,6,1995)", yl_commons_player_created = "1615351173", ["petz:werewolf_clan_idx"] = "1", yl_church = "return {[\"last_heal\"] = 1615504208, [\"last_death_portal\"] = 1668228099, [\"last_death\"] = {[\"x\"] = 6554, [\"y\"] = 2, [\"z\"] = 10487}}", ["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}", crafted = "47767", hud_state = "on", died = "73", ["stamina:level"] = "5", partychat = "party", jointime = "1615351173" } } ``` Log identifier ``` [MOD] yl_report log identifier = WjOSwYMsv8ld0ipiNwjwAgkkPFfqPL4W ``` Profiler save: ``` profile-20221121T045809.json_prettyEE ``` Status: ``` # Server: version: 5.6.1-yl | game: Minetest Game | uptime: 18h 27min 48s | max lag: 1.63s | clients (12/42): AliasAlreadyTaken, Bailiff, bodshfa, daydream, flux, madonna, niceride, Queenfire234312, Service, set, shanish, shanish2 ``` Teleport command: ``` /teleport xyz 1278 35 508 ``` Compass command: ``` /give_compass Construction WjOSwYMsv8ld0ipiNwjwAgkkPFfqPL4W D2691E 1278 35 508 ```
AliasAlreadyTaken was assigned by yourland-report 2022-11-21 04:58:09 +00:00
flux added the
1. kind/enhancement
label 2022-11-21 16:48:46 +00:00
flux added this to the flux's TODO list project 2022-11-21 17:04:20 +00:00
AliasAlreadyTaken was unassigned by flux 2022-11-21 17:04:25 +00:00
flux self-assigned this 2022-11-21 17:04:25 +00:00
Member

the idea here was to limit the # of actions per mapblock, not the amount of time they take.

the idea here was to limit the # of actions per mapblock, not the amount of time they take.

I don't like this idea. The only reasons machines need limitation is because of lag. If they arent laggy, there is no reason to slow them down. This would only be annoying.

I don't like this idea. The only reasons machines *need* limitation is because of lag. If they arent laggy, there is no reason to slow them down. This would only be annoying.
Member

I don't like this idea. The only reasons machines need limitation is because of lag. If they arent laggy, there is no reason to slow them down. This would only be annoying.

machines are one of the major generators of lag currently, which is why they need limitation. currently, mesecons_debug penalizes mesecons on how they affect the server globally. i think it'd be a better idea to limit the amount of computation locally against mapblocks. this would naturally limit them globally because players can only keep a small number of mapblocks loaded.

> I don't like this idea. The only reasons machines *need* limitation is because of lag. If they arent laggy, there is no reason to slow them down. This would only be annoying. machines are one of the major generators of lag currently, which is why they need limitation. currently, mesecons_debug penalizes mesecons on how they affect the server globally. i think it'd be a better idea to limit the amount of computation locally against mapblocks. this would naturally limit them globally because players can only keep a small number of mapblocks loaded.

Question. If this was implented, how many actions/sec would be allowed for one mapblock?

Question. If this was implented, how many actions/sec would be allowed for one mapblock?
Member

Question. If this was implented, how many actions/sec would be allowed for one mapblock?

we'd have to test to figure out what a good limit is.

> Question. If this was implented, how many actions/sec would be allowed for one mapblock? we'd have to test to figure out what a good limit is.

And what exactly would define an "action"?

And what exactly would define an "action"?
Member

And what exactly would define an "action"?

this would also have to be fleshed out.

also, note that this change is unlikely to be worked on anytime soon, i'm busy w/ a lot of other big projects. but i did want to make a note to consider this approach when i get back creating a job-scheduling framework for minetest.

> And what exactly would define an "action"? this would also have to be fleshed out. also, note that this change is unlikely to be worked on anytime soon, i'm busy w/ a lot of other big projects. but i did want to make a note to consider this approach when i get back creating a job-scheduling framework for minetest.

I think the current one is just fine and this one would just be more annoying.

I think the current one is just fine and this one would just be more annoying.
Sign in to join this conversation.
No Milestone
No Assignees
3 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#3093
No description provided.