From 3c841e885f3352fb86afa0db61ace72fa04dfb6a Mon Sep 17 00:00:00 2001 From: Dirk Sohler Date: Sun, 13 May 2018 22:00:30 +0200 Subject: [PATCH 1/2] fix and make the snow plates ABM configurable --- weather/settingtypes.txt | 8 ++++++++ weather/snow.lua | 36 +++++++++++++++++++----------------- 2 files changed, 27 insertions(+), 17 deletions(-) create mode 100644 weather/settingtypes.txt diff --git a/weather/settingtypes.txt b/weather/settingtypes.txt new file mode 100644 index 0000000..8ededa2 --- /dev/null +++ b/weather/settingtypes.txt @@ -0,0 +1,8 @@ +# Fast computer ABM +# +# The “fast computer ABM” controls if snow covers will be placed +# on the ground when `weather:snow` is set. This needs a better +# computer and should not be used if the performance drops too +# much. It is advised not to enable this on servers and it is +# disbaled by default +weather_fast_pc (Snow covers for fast PCs) bool false diff --git a/weather/snow.lua b/weather/snow.lua index 5c867bb..99aedb8 100644 --- a/weather/snow.lua +++ b/weather/snow.lua @@ -26,23 +26,25 @@ minetest.register_node("weather:snow_cover", { drop = {} }) ---[[ Enable this section if you have a very fast PC -minetest.register_abm({ - nodenames = {"group:crumbly", "group:snappy", "group:cracky", "group:choppy"}, - neighbors = {"default:air"}, - interval = 10.0, - chance = 80, - action = function (pos, node, active_object_count, active_object_count_wider) - if weather == "snow" then - if minetest.registered_nodes[node.name].drawtype == "normal" - or minetest.registered_nodes[node.name].drawtype == "allfaces_optional" then - local np = vector.add(pos, {x=0, y=1, z=0}) - if minetest.env:get_node_light(np, 0.5) == 15 - and minetest.env:get_node(np).name == "air" then - minetest.env:add_node(np, {name="weather:snow_cover"}) +-- Snow cover ABM when weather_fast_pc setting is set to `true` +if minetest.is_yes(minetest.settings:get_bool('weather_fast_pc')) then + minetest.log('action', '[weather] Loaded fast computer ABM (snow covers when weather:snow is set)') + minetest.register_abm({ + nodenames = {"group:crumbly", "group:snappy", "group:cracky", "group:choppy"}, + neighbors = {"default:air"}, + interval = 10.0, + chance = 80, + action = function (pos, node, active_object_count, active_object_count_wider) + if weather.type == "weather:snow" then + if minetest.registered_nodes[node.name].drawtype == "normal" + or minetest.registered_nodes[node.name].drawtype == "allfaces_optional" then + local np = vector.add(pos, {x=0, y=1, z=0}) + if minetest.env:get_node_light(np, 0.5) == 15 + and minetest.env:get_node(np).name == "air" then + minetest.env:add_node(np, {name="weather:snow_cover"}) + end end end end - end -}) -]] + }) +end From c2c0fce93aecb699c93ca3f105358992a8bd7208 Mon Sep 17 00:00:00 2001 From: Dirk Sohler Date: Sun, 13 May 2018 22:13:10 +0200 Subject: [PATCH 2/2] fix Travis `nused argument` errors --- weather/snow.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/weather/snow.lua b/weather/snow.lua index 99aedb8..92a039e 100644 --- a/weather/snow.lua +++ b/weather/snow.lua @@ -34,7 +34,7 @@ if minetest.is_yes(minetest.settings:get_bool('weather_fast_pc')) then neighbors = {"default:air"}, interval = 10.0, chance = 80, - action = function (pos, node, active_object_count, active_object_count_wider) + action = function (pos, node) if weather.type == "weather:snow" then if minetest.registered_nodes[node.name].drawtype == "normal" or minetest.registered_nodes[node.name].drawtype == "allfaces_optional" then