Post your modding questions here

[Anonymous_moose]

I am trying to make a wearable spacesuit that works with 3darmor and moontest. I have figured out how to get the breath to restore, but only cannot find out how to get the game to know if the space suit is being worn. any help would be much appreciated.

l guess you would need to search in the armour inventory slots.

I have tried that, but I really cannot find out how to manipulate the detached inventory slots.

[bobomb]

maybe using minetest.after(0, […] could work

that is what I ended up doing.

[cHyper]

Hi!

Are there any mods with achivements or Statistics elements.

Thanks

[rubenwardy]

See awards.

Or pilzadams stats.

[cHyper]

See awards.

Or pilzadams stats.

https://forum.minetest.net/viewtopic.php?f=11&t=8127
stats from PilzAdam seems to be a good mod, it functions but is outdated.

[stu]

I am trying to make a wearable spacesuit that works with 3darmor and moontest. I have figured out how to get the breath to restore, but only cannot find out how to get the game to know if the space suit is being worn. any help would be much appreciated.

l guess you would need to search in the armour inventory slots.

I have tried that, but I really cannot find out how to manipulate the detached inventory slots.

This should do it

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

local inv = player:get_inventory()
if inv:contains_item("armor", "your_mod_name:spacesuit") then
   --Do Stuff
end


[Esteban]

[Bones mod in Minetest_game]

I would like to know how to stop bones from disappearing when punched or getting removed when their emptied. In other words, I want to be able to collect bones.
I don't know what lines I must remove or rewrite.

+ Code from the version of bones mod I am currently using

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

-- Minetest 0.4 mod: bones
-- See README.txt for licensing and other information.

bones = {}

local function is_owner(pos, name)
   local owner = minetest.get_meta(pos):get_string("owner")
   if owner == "" or owner == name then
      return true
   end
   return false
end

bones.bones_formspec =
   "size[8,9]"..
   default.gui_bg..
   default.gui_bg_img..
   default.gui_slots..
   "list[current_name;main;0,0.3;8,4;]"..
   "list[current_player;main;0,4.85;8,1;]"..
   "list[current_player;main;0,6.08;8,3;8]"..
   default.get_hotbar_bg(0,4.85)

local share_bones_time = tonumber(minetest.setting_get("share_bones_time") or 1200)
local share_bones_time_early = tonumber(minetest.setting_get("share_bones_time_early") or (share_bones_time/4))

minetest.register_node("bones:bones", {
   description = "Bones",
   tiles = {
      "bones_top.png",
      "bones_bottom.png",
      "bones_side.png",
      "bones_side.png",
      "bones_rear.png",
      "bones_front.png"
   },
   paramtype2 = "facedir",
   groups = {dig_immediate=2},
   sounds = default.node_sound_dirt_defaults({
      footstep = {name="default_gravel_footstep", gain=0.5},
      dug = {name="default_gravel_footstep", gain=1.0},
   }),
   
   can_dig = function(pos, player)
      local inv = minetest.get_meta(pos):get_inventory()
      return is_owner(pos, player:get_player_name()) and inv:is_empty("main")
   end,
   
   allow_metadata_inventory_move = function(pos, from_list, from_index, to_list, to_index, count, player)
      if is_owner(pos, player:get_player_name()) then
         return count
      end
      return 0
   end,
   
   allow_metadata_inventory_put = function(pos, listname, index, stack, player)
      return 0
   end,
   
   allow_metadata_inventory_take = function(pos, listname, index, stack, player)
      if is_owner(pos, player:get_player_name()) then
         return stack:get_count()
      end
      return 0
   end,
   
   on_metadata_inventory_take = function(pos, listname, index, stack, player)
      local meta = minetest.get_meta(pos)
      if meta:get_inventory():is_empty("main") then
         minetest.remove_node(pos)
      end
   end,
   
   on_punch = function(pos, node, player)
      if(not is_owner(pos, player:get_player_name())) then
         return
      end
      
      local inv = minetest.get_meta(pos):get_inventory()
      local player_inv = player:get_inventory()
      local has_space = true
      
      for i=1,inv:get_size("main") do
         local stk = inv:get_stack("main", i)
         if player_inv:room_for_item("main", stk) then
            inv:set_stack("main", i, nil)
            player_inv:add_item("main", stk)
         else
            has_space = false
            break
         end
      end
      
      -- remove bones if player emptied them
      if has_space then
         minetest.remove_node(pos)
      end
   end,
   
   on_timer = function(pos, elapsed)
      local meta = minetest.get_meta(pos)
      local time = meta:get_int("time") + elapsed
      if time >= share_bones_time then
         meta:set_string("infotext", meta:get_string("owner").."'s old bones")
         meta:set_string("owner", "")
      else
         meta:set_int("time", time)
         return true
      end
   end,
})

local function may_replace(pos, player)
   local node_name = minetest.get_node(pos).name
   local node_definition = minetest.registered_nodes[node_name]

   -- if the node is unknown, we let the protection mod decide
   -- this is consistent with when a player could dig or not dig it
   -- unknown decoration would often be removed
   -- while unknown building materials in use would usually be left
   if not node_definition then
      -- only replace nodes that are not protected
      return not minetest.is_protected(pos, player:get_player_name())
   end

   -- allow replacing air and liquids
   if node_name == "air" or node_definition.liquidtype ~= "none" then
      return true
   end

   -- don't replace filled chests and other nodes that don't allow it
   local can_dig_func = node_definition.can_dig
   if can_dig_func and not can_dig_func(pos, player) then
      return false
   end

   -- default to each nodes buildable_to; if a placed block would replace it, why shouldn't bones?
   -- flowers being squished by bones are more realistical than a squished stone, too
   -- exception are of course any protected buildable_to
   return node_definition.buildable_to and not minetest.is_protected(pos, player:get_player_name())
end

minetest.register_on_dieplayer(function(player)
   if minetest.setting_getbool("creative_mode") then
      return
   end
   
   local player_inv = player:get_inventory()
   if player_inv:is_empty("main") and
      player_inv:is_empty("craft") then
      return
   end

   local pos = player:getpos()
   pos.x = math.floor(pos.x+0.5)
   pos.y = math.floor(pos.y+0.5)
   pos.z = math.floor(pos.z+0.5)
   local param2 = minetest.dir_to_facedir(player:get_look_dir())
   local player_name = player:get_player_name()
   local player_inv = player:get_inventory()

   if (not may_replace(pos, player)) then
      if (may_replace({x=pos.x, y=pos.y+1, z=pos.z}, player)) then
         -- drop one node above if there's space
         -- this should solve most cases of protection related deaths in which players dig straight down
         -- yet keeps the bones reachable
         pos.y = pos.y+1
      else
         -- drop items instead of delete
         for i=1,player_inv:get_size("main") do
            minetest.add_item(pos, player_inv:get_stack("main", i))
         end
         for i=1,player_inv:get_size("craft") do
            minetest.add_item(pos, player_inv:get_stack("craft", i))
         end
         -- empty lists main and craft
         player_inv:set_list("main", {})
         player_inv:set_list("craft", {})
         return
      end
   end
   
   minetest.set_node(pos, {name="bones:bones", param2=param2})
   
   local meta = minetest.get_meta(pos)
   local inv = meta:get_inventory()
   inv:set_size("main", 8*4)
   inv:set_list("main", player_inv:get_list("main"))
   
   for i=1,player_inv:get_size("craft") do
      local stack = player_inv:get_stack("craft", i)
      if inv:room_for_item("main", stack) then
         inv:add_item("main", stack)
      else
         --drop if no space left
         minetest.add_item(pos, stack)
      end
   end
   
   player_inv:set_list("main", {})
   player_inv:set_list("craft", {})
   
   meta:set_string("formspec", bones.bones_formspec)
   meta:set_string("owner", player_name)
   
   if share_bones_time ~= 0 then
      meta:set_string("infotext", player_name.."'s fresh bones")

      if share_bones_time_early == 0 or not minetest.is_protected(pos, player_name) then
         meta:set_int("time", 0)
      else
         meta:set_int("time", (share_bones_time - share_bones_time_early))
      end

      minetest.get_node_timer(pos):start(10)
   else
      meta:set_string("infotext", player_name.."'s bones")
   end
end)


Thanks in advance

[ExeterDad]

look for:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

      -- remove bones if player emptied them

Then change this:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

      -- remove bones if player emptied them
      if has_space then
         minetest.remove_node(pos)
      end

to this:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

     -- remove bones if player emptied them
      -- if has_space then
         -- minetest.remove_node(pos)
     --  end

[Anonymous_moose]

Thanks stu!

[Esteban]

Thanks ExeterDad! :D

[bobomb]

what is the best method to determine surface height given an x,z coordinate? is there something in the lua api? is there a preferred algorithm?

i found this but it was never merged? https://github.com/minetest/minetest/pull/640

thanks

[Hybrid Dog]

what is the best method to determine surface height given an x,z coordinate? is there something in the lua api? is there a preferred algorithm?

i found this but it was never merged? https://github.com/minetest/minetest/pull/640

thanks

ln which situation do you need to find out the surface height?
mapgen for example offers a heightmap mapgen_object

[bobomb]

i have solved this but the situation is not a mapgen situation. it is after the world is generated.

I ended up converting a function made for core that was never merged from: https://github.com/sapier/minetest/blob/c6f814b52a5e2b2f12c76215db3524571c56fe20/src/environment.cpp#L868

here it is:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

function get_surface(basepos, searchup, searchdown, walkable_only)

   local max = searchup + basepos.y
   if searchdown then basepos.y = basepos.y - searchdown end
   local last_node = minetest.get_node(basepos)
   local node = last_node
   local runpos = basepos

   local last_was_walkable = node.walkable

   while ( runpos.y < max and minetest.get_content_id(node.name) ~= c_air ) do
      runpos.y = runpos.y + 1
      last_node = node
      node = minetest.get_node(runpos)

      if not (walkable_only) then
         if minetest.get_content_id(last_node.name) ~= c_air
         and minetest.get_content_id(last_node.name) ~= c_ignore
         and minetest.get_content_id(node.name) == c_air then
            return runpos.y
         end
      else
         local is_walkable = node.walkable
         if last_was_walkable and not is_walkable then
            return runpos.y
         end
         last_was_walkable = is_walkable
      end
   end

   return basepos.y - 1
end


there are still some issues with this, like it returns max when nothing no air was detected, and it returns 0 when no ground was detected. also i don't know how to test for "is_walkable" in lua. how does one test a node for various group memberships, and is walkability a group or something else in the lua api?

[Hybrid Dog]

i have solved this but the situation is not a mapgen situation. it is after the world is generated.

I ended up converting a function made for core that was never merged from: https://github.com/sapier/minetest/blob/c6f814b52a5e2b2f12c76215db3524571c56fe20/src/environment.cpp#L868

here it is:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

function get_surface(basepos, searchup, searchdown, walkable_only)

   local max = searchup + basepos.y
   if searchdown then basepos.y = basepos.y - searchdown end
   local last_node = minetest.get_node(basepos)
   local node = last_node
   local runpos = basepos

   local last_was_walkable = node.walkable

   while ( runpos.y < max and minetest.get_content_id(node.name) ~= c_air ) do
      runpos.y = runpos.y + 1
      last_node = node
      node = minetest.get_node(runpos)

      if not (walkable_only) then
         if minetest.get_content_id(last_node.name) ~= c_air
         and minetest.get_content_id(last_node.name) ~= c_ignore
         and minetest.get_content_id(node.name) == c_air then
            return runpos.y
         end
      else
         local is_walkable = node.walkable
         if last_was_walkable and not is_walkable then
            return runpos.y
         end
         last_was_walkable = is_walkable
      end
   end

   return basepos.y - 1
end


there are still some issues with this, like it returns max when nothing no air was detected, and it returns 0 when no ground was detected. also i don't know how to test for "is_walkable" in lua. how does one test a node for various group memberships, and is walkability a group or something else in the lua api?

node.walkable is nil because node is from minetest.get_node(pos), which returns a table containing the name, param1 and param2 of the node at pos
this function should test if a node is walkable:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

-- is_walkable(minetest.get_node(pos).name)

local function is_walkable(name)
local data = minetest.registered_nodes[name]
if not data then
return false
end
return not (data.walkable == false)
end

walkable is not a group, it's something of the node definition, groups is something of the node definition, too
if you want to find out if a node is in a specific group, there's a function for it http://dev.minetest.net/minetest.get_item_group

[bobomb]

i knew that walkable code would not work, anyway here is a complete translation of sapier's function without any changes like "downsearch" which is redundant. (still need to deal with cases of no top found / no bottom found)

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

-- a lua rewrite of sapier's get_surface() proposal, see:
-- https://github.com/sapier/minetest/blob/c6f814b52a5e2b2f12c76215db3524571c56fe20/src/environment.cpp#L868
function get_surface(basepos, searchup, walkable_only)

   local max = searchup + basepos.y
   local last_node = minetest.get_node(basepos)
   local node = last_node
   local runpos = basepos

   local last_was_walkable = minetest.registered_nodes[node.name].walkable

   while ( runpos.y < max and minetest.get_content_id(node.name) ~= c_air ) do
      runpos.y = runpos.y + 1
      last_node = node
      node = minetest.get_node(runpos)

      if not (walkable_only) then
         if minetest.get_content_id(last_node.name) ~= c_air
         and minetest.get_content_id(last_node.name) ~= c_ignore
         and minetest.get_content_id(node.name) == c_air then
            return runpos.y
         end
      else
         local is_walkable = minetest.registered_nodes[node.name].walkable
         if last_was_walkable and not is_walkable then
            return runpos.y
         end
         last_was_walkable = is_walkable
      end
   end

   return basepos.y - 1
end


[rubenwardy]

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

local data = minetest.registered_nodes[name]if not data then
    return false
end
return not (data.walkable == false)

Would be simpler as:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

return data and data.walkable

Table look ups can be costly. minetest.get_node can also be costly. get_node will return { name = "ignore" } is the MapBlock hasn't been loaded yet. Node that there may be stacked worlds - floating lands etc at 30000 (in the future) and you may be in a cave.

[bobomb]

Thanks Ruben. I am going to revise the function to handle caves and skylands, one way or another. My first approach was to make it relative to the player, but anyway that is on the to do list...

is Hybrid Dog's function necessary or can I just do

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

return minetest.registered_nodes[node.name].walkable


i know node name exists because that's what's being tested.

[rubenwardy]

There may be unknown nodes in the world, this will cause the game to crash.

[Hybrid Dog]

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

local data = minetest.registered_nodes[name]if not data then
    return false
end
return not (data.walkable == false)

[/quote

Would be simpler as:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

return data and data.walkable

l'm not sure if this works.
But you could short it this way:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

local data = minetest.registered_nodes[name]
return data and data.walkable ~= false

[bobomb]

I rewrote the function to scan from the top down which is much simpler and deals with caves and overhangs at least in a basic way:
https://gist.github.com/bobombolo/6127b153228344eb037a

[SAMIAMNOT]

How do you spawn nodes and/or mobs? I tried modding the More Mesecons mod and changed the add_item function to add_node but that did not work for the Mobs Redo mob NPCs.

[Nathan.S]

Doubtless I'm just an idiot but can somebody tell me what is wrong with this?

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

if inv:contains_item('fuel', 'group:tree') then

It works just fine for non-groups, but I want to support multiple fuels as inputs in a smoker I'm making. I suppose I could just do a chain of if statements but a group would be quicker and easier.

[kaeza]

Note: UNTESTED:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

local function is_item_in_group(itemname, group)
  local def = minetest.registered_items[name]
  if not def then return end
  return (def.groups and (def.groups[group] or 0)>0)
end
-- In your code (this assumes "fuel" inv only has a single slot):
-- ...
local item = inv:get_stack("fuel", 1)
local fuel_groups = { "tree", "wood", "oil", "radiactive" } -- Groups considered "fuel"
if item and (not item:is_empty()) then
  for _, g in ipairs(fuel_groups) do
    if is_item_in_group(item:get_name(), g) then
      -- We found some fuel. Do something with it.
      return
    end
  end
end
-- ...


Adapt as necessary.

Of course, the proper solution would be to implement group resolution or whatever in `contains_item`.

[Nathan.S]

Thank you so much kaeza, this looks like it should work. I just want people to be able to use logs, so I'll just be using the tree group. I tried just making a table and doing a if in table, which I know would work in python, but unfortunately LUA doesn't have that same feature.

[minetestcr]

Hello!

Is there a way to have one user in creative mode while the others are not? It would be the server admin in creative mode, and the players in survival.

thanks.

[benrob0329]

Unified Inventory has a creative priv, that could work.

[minetestcr]

Unified Inventory has a creative priv, that could work.

Thanks benrob0329.

[BrunoMine]

how to identify the lag of a client. I do not care only server lag, but each client individually.

[Semmett9]

How to activate a file if a mod is installed?

Is they a way to activate a file or bit of code if a mod is installed.

I have just mad a mod called "Beer_Test" and I’m currently working on a new mod "Farm_Test". Is they a way for me to add a line of code which looks for a mod called "Beer_Test"; and if it finds it will then activate some code which connects the two mods together.

Many Thanks

[kaeza]

You need to add a file named `depends.txt` to the `farm_test` mod (if it doesn't already have one), and add this line to the file:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

beer_test?

This ensures the `beer_test` mod gets loaded first if found, but the mod is not required (i.e. it is optional).

Then, in `farm_test/init.lua` (or wherever appropriate), you use code like this:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

if minetest.get_modpath("beer_test") then
  -- beer_test is installed. Add some glue code, extra items, etc. here.
end