Factorio Forums

Posted: **Fri Nov 03, 2017 11:51 am**

Hi all!
How to make this function?

I have a

table = {
{name = "name-01", probability = 100},
{name = "name-02", probability = 90},
{name = "name-03", probability = 20},
{name = "name-04", probability = 20},
...
}

And I want to get randomly element from this table.
The first step is easy, I need to get summary of all probabilities and then I can get

Code: Select all

number =math.rnd(probability_summ(table))

How to get the element from this table?

Posted: **Fri Nov 03, 2017 12:15 pm**

create the cumulative sum and then pick a number between 0 and the sum of the original:

Code: Select all

table = {
{name = "name-01", probability = 100, cumsum = 100},
{name = "name-02", probability = 90, cumsum = 190},
{name = "name-03", probability = 20, cumsum = 210},
{name = "name-04", probability = 20, , cumsum = 230},
...
}

number =math.rnd(probability_summ(table));

for i,v in ipairs(a) do 
    if(number < v.cumsum) then result = i; break; end
end
// or binary search instead.

Posted: **Fri Nov 03, 2017 2:09 pm**

Propability is usually expressed as a value between 0 and 100%. Your tables total probability is >100. So you should recalculate the values to (42,40,9,9) to make it easier to tell how probable each entry occurs. (Unless your intent is to have the possibility for several "entries" to occur at the same time in which case the above solution is wrong). If you want to keep the numeric values as in your example then they're not probability values but weight values and the variable should be renamed.

This...is ofc all assuming you care about making your code readable to other people

Posted: **Fri Nov 03, 2017 2:46 pm**

eradicator wrote:Propability is usually expressed as a value between 0 and 100%. Your tables total probability is >100.

Probability can be written by "permille", then total must be 1000.

But you are right, it was all about weights. In this case weights are better, I can use integer only, the summary will be always 100%, not 'fast 100' or 'a bit more then 100' caused by fraction rounding.

Posted: **Fri Nov 03, 2017 2:56 pm**

We do something similar for team production script:
The definition

Code: Select all

  global.challange_type_probabilities =
  {
    {probability = 14, value = "production"},
    {probability = 2, value = "research"},
    {probability = 14, value = "shopping_list"},
    {probability = 8, value = "input_output"},
  }

And how to choose

Code: Select all

function select_from_probability_table(probability_table)
  local roll_max = 0
  for _, item in pairs(probability_table) do
    roll_max = roll_max + item.probability
  end

  local roll_value = math.random(0, roll_max - 1)
  for _, item in pairs(probability_table) do
    roll_value = roll_value - item.probability
    if (roll_value < 0) then
      return item.value
    end
  end
end

Posted: **Fri Nov 03, 2017 4:38 pm**

darkfrei wrote:
eradicator wrote:Propability is usually expressed as a value between 0 and 100%. Your tables total probability is >100.
Probability can be written by "permille", then total must be 1000.

Just making sure you actually wanted weight and not per-item-probability. Because (like Klonans example too) if i read "probability=14" i first assume that "that thing" has 14% chance to happen, and not 36%

.
If you need to do this really fast you might try pre-caching the result:

Code: Select all

local guys = {
  {who='beggar' , weight=20},
  {who='burglar', weight=50},
  {who='bandit' , weight=10},
  {who='barman' , weight=10},
  }

Code: Select all

local brawl = {}
for k,guy in iparis(guys) do
  for i=1,guy.weight do
     brawl[#brawl+1] = k
    end
  end

local injuries = #brawl

local function fight()
  return guys[brawl[math.random(injuries)]]
  end

Code: Select all

local winner = fight()

So every roll requires just two table lookups. Can be reduced to one lookup if you put the 'guy's name directly into the brawl table but then it costs more memory.

Posted: **Wed Jan 03, 2018 5:03 pm**

Hey Guys!

I please need your help to implement this logic in my mod - Bio Industries.

What I do in the mod, is have one have the ability to plant seedlings that will grow into trees.
Since different terrains grow different type of trees and the trees have different probabilities of growing in each terrain, I need to check the terrain you plant the seedling in agains ta table and then randomly choose a tree to grow there, based on the trees weight/probability to grow in that type of terrain.

Currently I'm just doing a random select, but thanks to Darkfrei, I actually have a table to each terrain and the trees with weights/probablilitys per terrain. See thread here.

So, I've attached my tree table. (trees-and-terrains.lua)

I need to basically replace the code below in the "control_tree.lua" file, with the stuff from the other link you sent me... seems so simple

Code

Code: Select all


local function get_random_from_table (table)
   -- compressed table if some elements was deleted and we have {[1] = value1, [3] = value3}
   local compressed_table = {}
   for i, value in pairs (table) do
      if value then 
         compressed_table[#compressed_table+1] = value
      end
   end
   return compressed_table[math.random(1,#compressed_table)]
end



local function get_random_tree_for_terrain (terrain_name) -- text
  
   local terrains = {}
   --Vanilla
	terrains["grass-1"] = {
	  name = "grass-1",
	  trees = {
	   "tree-02",   
	   "tree-03",
	   "tree-04",
	   "tree-05"
	  }
	}
	terrains["grass-2"] = {
	  name = "grass-2",
	  trees = {
		"tree-01",
		"tree-02",
		"tree-03",
		"tree-04",
		"tree-05",
		"tree-07",
		"tree-08"
	  }
	}
	terrains["grass-3"] = {
	  name = "grass-3",
	  trees = {     
		"tree-01",
		"tree-03",
		"tree-04",
		"tree-05",
		"tree-07"
	  }
	}
	terrains["grass-4"] = {
	  name = "grass-4",
	  trees = {     
		"tree-01",
		"tree-03",
		"tree-04",
		"tree-05",
		"tree-07"
	  }
	}
	terrains["sand-1"] = {
	  name = "sand-1",
	  trees = {
		"tree-07",
		"tree-09-brown",
	  }
	}
	terrains["sand-2"] = {
	  name = "sand-2",
	  trees = {
		"tree-01",
		"tree-07",
		"tree-08-brown",
		"tree-09-brown",
		"tree-09"
	  }
	}
	terrains["sand-3"] = {
	  name = "sand-3",
	  trees = {
		"tree-07",
		"tree-08-brown",
		"tree-09-brown",
	  }
	}
	terrains["dry-dirt"] = {
	  name = "dry-dirt",
	  trees = {
		"tree-01",
		"tree-07",
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-1"] = {
	  name = "dirt-1",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-2"] = {
	  name = "dirt-2",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-3"] = {
	  name = "dirt-3",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-4"] = {
	  name = "dirt-4",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-5"] = {
	  name = "dirt-5",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-6"] = {
	  name = "dirt-6",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["dirt-7"] = {
	  name = "dirt-7",
	  trees = {
		"tree-08-brown",
		"tree-09-brown"
	  }
	}
	terrains["red-desert-0"] = {
	  name = "red-desert-0",
	  trees = {
		"tree-09-red"
	  }
	}
	terrains["red-desert-1"] = {
	  name = "red-desert-1",
	  trees = {
		"tree-09-red"
	  }
	}
	terrains["red-desert-2"] = {
	  name = "red-desert-2",
	  trees = {
		"tree-09-red"
	  }
	}
	terrains["red-desert-3"] = {
	  name = "red-desert-3",
	  trees = {
		"tree-09-red"
	  }
	}

   for i, terrain in pairs (terrains) do
      if (terrain.name == terrain_name) then 
         return get_random_from_table (terrain.trees)
      end
   end
   
end

I'd appreciate it if someone could help me out a little with this, it's just outside of my coding skills. And also possibly looking at the rest of the code, making sure I'm not messing something up...

Thanks!

Posted: **Wed Jan 03, 2018 8:03 pm**

Code: Select all

terrains = require("trees-and-terrains")

function summ_weight (tabl)
  local summ = 0
  for i, tree_weights in pairs (tabl) do
    if (type (tree_weights) == "table") and tree_weights.weight then
      summ = summ + tree_weights.weight
    end
  end
  return summ
end

function tree_from_max_index_tabl (max_index, tabl)
  local rnd_index = math.random (max_index)
  for tree_name, tree_weights in pairs (tabl) do
    if (type (tree_weights) == "table") and tree_weights.weight then
      rnd_index = rnd_index - tree_weights.weight
      if rnd_index <= 0 then
        return tree_name
      end
    end
  end
  return nil
end

function random_tree (surface, position)
  local tile = surface.get_tile(position.x, position.y)
  local tile_name = tile.name
  if is_value_as_index_in_table (tile_name, terrains) then
    local trees_table = terrains[tile_name]
    local max_index = summ_weight(trees_table)
    return tree_from_max_index_tabl (max_index, trees_table)
  end
end

So, you can call this functions like here (you are need surface and position of tile or entity with your seed):

Code: Select all

local tree_name = random_tree (surface, position)
if tree_name then
  local can_be_placed = surface.can_place_entity{name=tree_name, position=position, force = "neutral"}
  if can_be_placed then
    local tree = surface.create_entity{name=tree_name, position=position, force = "neutral"}
  end
end

Posted: **Sat Jan 06, 2018 8:04 am**

I'm still not sure what exactly your problem with the implementation is, it sounds very straight-foward to me.
If you have a terrain-to-tree-weight mapping table like:

Code: Select all

local tile_to_tree = {
  ["red-desert-3"] = {
    {5 ,"tree-01"},
    {10,"tree-02"},
    {20,"tree-03"},
  }
}

You can easily fetch the tree map for a given tile:

Code: Select all

local tree_weight_map = tile_to_tree[surface.get_tile(position).name]

and then use the fight() function from my above post.
Also with a static map like this you might consider including the total weight into the premade table instead of calculating it each time.
(Your current trees-and-terrains.lua table seems to have a lot of redundancy, which imho just makes it more difficult to read while not gaining any benefit by including tile_name and tree_name for each tree.)

EDIT:
Used some regex magic to reformat the table to an imho more readable format (warning, forum does not use monospace formatting):

tiles and trees

Code: Select all

-- Table of terrains and trees found in them (weighted)
   local terrains = {}
   
   
terrains["dirt-6"] = {
  {    24,"dry-hairy-tree"},
  {    42,"dry-tree"},
  {   221,"tree-01"},
  { 24234,"tree-02"},
  {179041,"tree-02-red"},
  {     2,"tree-03"},
  { 69753,"tree-04"},
  {  1068,"tree-05"},
  {    36,"tree-06"},
  {     8,"tree-06-brown"},
  {    16,"tree-07"},
  {    16,"tree-08"},
  {    32,"tree-08-brown"},
  {   558,"tree-09"},
  {    66,"tree-09-brown"},
  {   165,"tree-09-red"},
}, 
terrains["dirt-7"] = {
  {    30,"dry-hairy-tree"},
  {    38,"dry-tree"},
  {   577,"tree-01"},
  { 98661,"tree-02"},
  { 75918,"tree-02-red"},
  {     2,"tree-03"},
  { 66040,"tree-04"},
  {  3064,"tree-05"},
  {    49,"tree-06"},
  {    33,"tree-06-brown"},
  {    28,"tree-07"},
  {     6,"tree-08"},
  {    11,"tree-08-brown"},
  {   210,"tree-09"},
  {   253,"tree-09-brown"},
  {   183,"tree-09-red"},
}, 
terrains["dirt-5"] = {
  {    46,"dry-hairy-tree"},
  {    51,"dry-tree"},
  {   100,"tree-01"},
  {  3358,"tree-02"},
  {187794,"tree-02-red"},
  {     3,"tree-03"},
  { 64545,"tree-04"},
  {   193,"tree-05"},
  {    28,"tree-06"},
  {    23,"tree-06-brown"},
  {     2,"tree-07"},
  {    27,"tree-08"},
  {   669,"tree-09"},
  {   106,"tree-09-brown"},
  {    55,"tree-09-red"},
}, 
terrains["dirt-3"] = {
  {   196,"dry-hairy-tree"},
  {   218,"dry-tree"},
  {    22,"tree-01"},
  {   851,"tree-02"},
  { 82425,"tree-02-red"},
  {    12,"tree-03"},
  { 35174,"tree-04"},
  {   432,"tree-05"},
  {    58,"tree-06"},
  {    30,"tree-06-brown"},
  {    80,"tree-07"},
  {    20,"tree-08"},
  {    33,"tree-08-brown"},
  {   260,"tree-09"},
  {  1127,"tree-09-brown"},
  {   765,"tree-09-red"},
}, 
terrains["sand-1"] = {
  { 12157,"dry-hairy-tree"},
  { 12561,"dry-tree"},
  {   555,"tree-01"},
  { 60844,"tree-02"},
  { 57696,"tree-02-red"},
  {   415,"tree-03"},
  { 38098,"tree-04"},
  { 17776,"tree-05"},
  { 69212,"tree-06"},
  { 61028,"tree-06-brown"},
  { 11335,"tree-07"},
  { 28216,"tree-08"},
  { 31660,"tree-08-brown"},
  {   218,"tree-09"},
  {146383,"tree-09-brown"},
  {126450,"tree-09-red"},
}, 
terrains["sand-3"] = {
  {   440,"dry-hairy-tree"},
  {   479,"dry-tree"},
  {   362,"tree-02"},
  {   242,"tree-02-red"},
  {     2,"tree-03"},
  {    25,"tree-04"},
  {     5,"tree-05"},
  {  2452,"tree-06"},
  {  2035,"tree-06-brown"},
  {  2573,"tree-07"},
  {   778,"tree-08"},
  {   997,"tree-08-brown"},
  { 24871,"tree-09-brown"},
  { 23289,"tree-09-red"},
}, 
terrains["dirt-2"] = {
  {   537,"dry-hairy-tree"},
  {   582,"dry-tree"},
  {     8,"tree-01"},
  {  1144,"tree-02"},
  { 13364,"tree-02-red"},
  {    71,"tree-03"},
  { 10320,"tree-04"},
  {   908,"tree-05"},
  {    68,"tree-06"},
  {    27,"tree-06-brown"},
  {   501,"tree-07"},
  {    13,"tree-08"},
  {    48,"tree-08-brown"},
  {    37,"tree-09"},
  {  5694,"tree-09-brown"},
  {  4715,"tree-09-red"},
}, 
terrains["dirt-1"] = {
  {   643,"dry-hairy-tree"},
  {   614,"dry-tree"},
  {   403,"tree-02"},
  {   609,"tree-02-red"},
  {   755,"tree-04"},
  {    11,"tree-05"},
  {     5,"tree-06"},
  {     2,"tree-06-brown"},
  {   770,"tree-07"},
  {    13,"tree-08"},
  {    39,"tree-08-brown"},
  {     4,"tree-09"},
  { 10364,"tree-09-brown"},
  {  8331,"tree-09-red"},
}, 
terrains["sand-2"] = {
  {   259,"dry-hairy-tree"},
  {   300,"dry-tree"},
  {   160,"tree-02"},
  {   214,"tree-02-red"},
  {    15,"tree-04"},
  {     4,"tree-05"},
  {  3931,"tree-06"},
  {  3176,"tree-06-brown"},
  {  1481,"tree-07"},
  {   583,"tree-08"},
  {   658,"tree-08-brown"},
  { 16053,"tree-09-brown"},
  { 13349,"tree-09-red"},
}, 
terrains["dry-dirt"] = {
  {   781,"dry-hairy-tree"},
  {   800,"dry-tree"},
  {   521,"tree-02"},
  {  2271,"tree-02-red"},
  {  1455,"tree-04"},
  {     8,"tree-05"},
  {  4513,"tree-06"},
  {  3923,"tree-06-brown"},
  {   748,"tree-07"},
  {  2190,"tree-08"},
  {  2021,"tree-08-brown"},
  {     5,"tree-09"},
  {  7876,"tree-09-brown"},
  {  7123,"tree-09-red"},
}, 
terrains["dirt-4"] = {
  {   257,"dry-hairy-tree"},
  {   280,"dry-tree"},
  {   218,"tree-02"},
  { 24554,"tree-02-red"},
  {  8460,"tree-04"},
  {     6,"tree-05"},
  {   659,"tree-06"},
  {   698,"tree-06-brown"},
  {   409,"tree-07"},
  {   603,"tree-08"},
  {   639,"tree-08-brown"},
  {    61,"tree-09"},
  {  4361,"tree-09-brown"},
  {  3611,"tree-09-red"},
}, 
terrains["red-desert-1"] = {
  {   399,"dry-hairy-tree"},
  {   415,"dry-tree"},
  {   299,"tree-02"},
  {   260,"tree-02-red"},
  {   158,"tree-04"},
  {     5,"tree-05"},
  {  1008,"tree-06"},
  {   806,"tree-06-brown"},
  {   842,"tree-07"},
  {   684,"tree-08"},
  {   389,"tree-08-brown"},
  {  9629,"tree-09-brown"},
  {  8543,"tree-09-red"},
}, 
terrains["grass-1"] = {
  {   167,"dry-hairy-tree"},
  {   170,"dry-tree"},
  {  2770,"tree-01"},
  {128833,"tree-02"},
  {  3855,"tree-02-red"},
  {  4225,"tree-03"},
  { 55047,"tree-04"},
  {128934,"tree-05"},
  {   333,"tree-06"},
  {   219,"tree-06-brown"},
  {   115,"tree-07"},
  {   231,"tree-08"},
  {   269,"tree-08-brown"},
  {    10,"tree-09"},
  {  1379,"tree-09-brown"},
  {   698,"tree-09-red"},
}, 
terrains["grass-3"] = {
  {   111,"dry-hairy-tree"},
  {    89,"dry-tree"},
  {  3042,"tree-01"},
  {213831,"tree-02"},
  {  1221,"tree-02-red"},
  {  4680,"tree-03"},
  { 67866,"tree-04"},
  {147689,"tree-05"},
  {    41,"tree-06"},
  {    29,"tree-06-brown"},
  {    28,"tree-07"},
  {    70,"tree-08"},
  {    42,"tree-08-brown"},
  {   300,"tree-09-brown"},
  {   125,"tree-09-red"},
}, 
terrains["grass-4"] = {
  {    62,"dry-hairy-tree"},
  {    44,"dry-tree"},
  {  1386,"tree-01"},
  {328767,"tree-02"},
  { 14228,"tree-02-red"},
  {  1289,"tree-03"},
  {111457,"tree-04"},
  { 53084,"tree-05"},
  {    43,"tree-06"},
  {    99,"tree-06-brown"},
  {     6,"tree-07"},
  {    15,"tree-08"},
  {    14,"tree-08-brown"},
  {    80,"tree-09"},
  {   166,"tree-09-brown"},
  {    90,"tree-09-red"},
}, 
terrains["red-desert-2"] = {
  {   108,"dry-hairy-tree"},
  {   140,"dry-tree"},
  {    78,"tree-02"},
  {    59,"tree-02-red"},
  {    17,"tree-04"},
  {   647,"tree-06"},
  {   673,"tree-06-brown"},
  {   365,"tree-07"},
  {   363,"tree-08"},
  {   265,"tree-08-brown"},
  {  4827,"tree-09-brown"},
  {  3270,"tree-09-red"},
}, 
terrains["red-desert-0"] = {
  {    28,"dry-hairy-tree"},
  {    13,"dry-tree"},
  {   323,"tree-01"},
  { 78399,"tree-02"},
  { 54323,"tree-02-red"},
  {    82,"tree-03"},
  { 49164,"tree-04"},
  {  4627,"tree-05"},
  {    25,"tree-06"},
  {     4,"tree-06-brown"},
  {    18,"tree-08"},
  {   202,"tree-09"},
  {    26,"tree-09-brown"},
  {    16,"tree-09-red"},
}, 
terrains["grass-2"] = {
  {    26,"dry-hairy-tree"},
  {    17,"dry-tree"},
  {   941,"tree-01"},
  {111956,"tree-02"},
  {  8561,"tree-02-red"},
  {   572,"tree-03"},
  { 37335,"tree-04"},
  { 23569,"tree-05"},
  {    61,"tree-06"},
  {    40,"tree-06-brown"},
  {     4,"tree-07"},
  {   116,"tree-08"},
  {    25,"tree-08-brown"},
  {    15,"tree-09"},
  {    59,"tree-09-brown"},
  {    54,"tree-09-red"},
}, 
terrains["red-desert-3"] = {
  {   185,"dry-hairy-tree"},
  {   165,"dry-tree"},
  {    99,"tree-02"},
  {   597,"tree-02-red"},
  {   225,"tree-04"},
  {  1104,"tree-06"},
  {  1019,"tree-06-brown"},
  {   197,"tree-07"},
  {   441,"tree-08"},
  {   343,"tree-08-brown"},
  {     6,"tree-09"},
  {  1648,"tree-09-brown"},
  {  1505,"tree-09-red"},
},

Factorio Forums

Random with probability

Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability

Re: Random with probability