← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~7010622-q/widelands/topple-production-logic-4 into lp:widelands

 

toptopple has proposed merging lp:~7010622-q/widelands/topple-production-logic-4 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~7010622-q/widelands/topple-production-logic-4/+merge/309656

These are the adjustment time values for weapon and armour production sites (all races) which were changed recently for more efficient use of their schemes. The new values are required to compensate increased production speed in the "mixed resources" cases, which must be considered standard. Mixed resources means there are not all resources supplied for the entire set of products the economy needs. 

Though these changes may appear voluminous to some, actually they are still rather weak. I test-played them all and they render nicely. With only 2 houses "weapon-smith" (and 2 for armour) you receive a constant flow of equipment and can run your training camp at 100% if all raw resources are supplied.

The effects are two-fold:
1) production time values are lifted (e.g.t o allow for resupplies of raw material)
2) times of products are diversified according to their value. That means for a higher product more time is invested than for a lower. Values in seconds.

Site : Product          Old    New      Change  Factor
------------------------------------------------------
BA AxeWo : Axe		57	73	16	0.28
BA AxeWo : Axe-Sharp	57	76	19	0.33
BA AxeWo : Axe-Broad	57	79	22	0.38
				
BA Warmill : Axe	57	73	16	0.28
BA Warmill : Axe-Sharp	57	76	19	0.33
BA Warmill : Axe-Broad	57	79	22	0.38
BA Warmill : Axe-Bronze	57	82	25	0.44
BA Warmill : Axe-Battle	57	87	30	0.53
BA Warmill : Axe-Warrio	57	92	35	0.61
				
BA Helms : Helmet	67	80	13	0.19
BA Helms : Mask		77	90	15	0.18
BA Helms : Warhelm	87	100	17	0.15

EM Weapons : Wood Spear	50	64	14	0.28
EM Weapons : Spear	77	102	25	0.32
EM Weapons : Adv. Spear	77	110	33	0.43
EM Weapons : Hv. Spear	77	115	38	0.49
EM Weapons : War-Spear	77	120	43	0.56
				
EM Armors : Helmet	67	80	13	0.19
EM Armors : Armor	77	90	13	0.17
EM Armors : Chain Armor	77	108	31	0.40
EM Armors : Guild Armor	77	115	38	0.49
				
AT Weapons : Tri Light	50	64	14	0.28
AT Weapons : Tri Long	77	102	25	0.32
AT Weapons : Tri Steel	77	110	33	0.43
AT Weapons : Tri Double	77	115	38	0.49
AT Weapons : Tri H-D	77	120	43	0.56
				
AT Armors : Shld Steel	67	90	23	0.34
AT Armors : Shld Adv.	77	110	33	0.43

-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~7010622-q/widelands/topple-production-logic-4 into lp:widelands.
=== modified file 'data/tribes/buildings/productionsites/atlanteans/armorsmithy/init.lua'
--- data/tribes/buildings/productionsites/atlanteans/armorsmithy/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/atlanteans/armorsmithy/init.lua	2016-10-31 07:27:05 +0000
@@ -132,10 +132,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a steel shield because ...
          descname = _"forging a steel shield",
          actions = {
+            -- time total: 90
             "return=skipped unless economy needs shield_steel",
-            "sleep=32000",
+            "sleep=45000",  -- +13 enlarge
             "consume=iron:2 coal:2",
-            "animate=working 35000",
+            "animate=working 45000",  -- +10 enlarge
             "produce=shield_steel"
          }
       },
@@ -143,10 +144,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging an advanced shield because ...
          descname = _"forging an advanced shield",
          actions = {
+            -- time total: 110
             "return=skipped unless economy needs shield_advanced",
             "consume=iron:2 coal:2 gold",
-            "sleep=32000",
-            "animate=working 45000",
+            "sleep=50000",  -- +18 enlarge
+            "animate=working 60000",  -- +15 enlarge
             "produce=shield_advanced"
          }
       },

=== modified file 'data/tribes/buildings/productionsites/atlanteans/weaponsmithy/init.lua'
--- data/tribes/buildings/productionsites/atlanteans/weaponsmithy/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/atlanteans/weaponsmithy/init.lua	2016-10-31 07:27:05 +0000
@@ -72,11 +72,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a light trident because ...
          descname = _"forging a light trident",
          actions = {
+            -- time total: 64
             "return=skipped unless economy needs trident_light",
-            "sleep=20000",
+            "sleep=27000",    -- +7 enlarge
             "consume=iron planks",
             "play_sound=sound/smiths smith 192",
-            "animate=working 21000",
+            "animate=working 28000",  -- +7 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=trident_light"
@@ -86,11 +87,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a long trident because ...
          descname = _"forging a long trident",
          actions = {
+            -- time total: 102
             "return=skipped unless economy needs trident_long",
             "consume=iron coal planks",
-            "sleep=32000",
+            "sleep=47000",  -- +15 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 46000",  -- +10 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=trident_long"
@@ -100,11 +102,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a steel trident because ...
          descname = _"forging a steel trident",
          actions = {
+            -- time total: 110
             "return=skipped unless economy needs trident_steel",
             "consume=iron:2 coal planks",
-            "sleep=32000",
+            "sleep=50000",  -- +18 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 51000",  -- +15 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=trident_steel"
@@ -114,11 +117,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a double trident because ...
          descname = _"forging a double trident",
          actions = {
+            -- time total: 115
             "return=skipped unless economy needs trident_double",
             "consume=iron coal:2 planks gold",
-            "sleep=32000",
+            "sleep=55000",  -- +23 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 51000",  -- +15 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=trident_double"
@@ -128,11 +132,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a heavy double trident because ...
          descname = _"forging a heavy double trident",
          actions = {
-            "return=skipped unless economy needs trident_heavy_double",
+             -- time total: 120
+           "return=skipped unless economy needs trident_heavy_double",
             "consume=iron:2 coal:2 planks gold",
-            "sleep=32000",
+            "sleep=55000",  -- +23 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 56000",  -- +20 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=trident_heavy_double"

=== modified file 'data/tribes/buildings/productionsites/barbarians/ax_workshop/init.lua'
--- data/tribes/buildings/productionsites/barbarians/ax_workshop/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/barbarians/ax_workshop/init.lua	2016-10-31 07:27:05 +0000
@@ -74,11 +74,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging an ax because ...
          descname = _"forging an ax",
          actions = {
+            -- time total: 73
             "return=skipped unless economy needs ax",
-            "sleep=26000",
+            "sleep=36000", -- +10 enlarge
             "consume=coal iron",
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 28000",  -- +6 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax"
@@ -88,11 +89,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a sharp ax because ...
          descname = _"forging a sharp ax",
          actions = {
+            -- time total: 76
             "return=skipped unless economy needs ax_sharp",
             "consume=coal iron:2",
-            "sleep=26000",
+            "sleep=37000", -- +11 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 30000", -- +8 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_sharp"
@@ -102,11 +104,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a broad ax because ...
          descname = _"forging a broad ax",
          actions = {
+            -- time total: 79
             "return=skipped unless economy needs ax_broad",
             "consume=coal:2 iron:2",
-            "sleep=26000",
+            "sleep=38000", -- +12 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 32000", -- +10 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_broad"

=== modified file 'data/tribes/buildings/productionsites/barbarians/helmsmithy/init.lua'
--- data/tribes/buildings/productionsites/barbarians/helmsmithy/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/barbarians/helmsmithy/init.lua	2016-10-31 07:27:05 +0000
@@ -77,10 +77,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a helmet because ...
          descname = _"forging a helmet",
          actions = {
+            -- time total: 80
             "return=skipped unless economy needs helmet",
-            "sleep=32000",
+            "sleep=40000", -- +8 enlarge
             "consume=coal iron",
-            "animate=working 35000",
+            "animate=working 40000", -- +5 enlarge
             "produce=helmet"
          }
       },
@@ -88,10 +89,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a mask because ...
          descname = _"forging a mask",
          actions = {
+            -- time total: 90
             "return=skipped unless economy needs helmet_mask",
             "consume=coal iron:2",
-            "sleep=32000",
-            "animate=working 45000",
+            "sleep=40000", -- +8 enlarge
+            "animate=working 50000", -- +5 enlarge
             "produce=helmet_mask"
          }
       },
@@ -99,10 +101,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a warhelm because ...
          descname = _"forging a warhelm",
          actions = {
+            -- time total: 100
             "return=skipped unless economy needs helmet_warhelm",
             "consume=coal gold iron:2",
-            "sleep=32000",
-            "animate=working 55000",
+            "sleep=40000", -- +8 enlarge
+            "animate=working 60000", -- +5 enlarge
             "produce=helmet_warhelm"
          }
       },

=== modified file 'data/tribes/buildings/productionsites/barbarians/warmill/init.lua'
--- data/tribes/buildings/productionsites/barbarians/warmill/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/barbarians/warmill/init.lua	2016-10-31 07:27:05 +0000
@@ -81,11 +81,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging an ax because ...
          descname = _"forging an ax",
          actions = {
+            -- time total: 73
             "return=skipped unless economy needs ax",
-            "sleep=26000",
+            "sleep=36000", -- +10 enlarge
             "consume=coal iron",
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 28000",  -- +6 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax"
@@ -95,11 +96,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a sharp ax because ...
          descname = _"forging a sharp ax",
          actions = {
+            -- time total: 76
             "return=skipped unless economy needs ax_sharp",
             "consume=coal iron:2",
-            "sleep=26000",
+            "sleep=37000", -- +11 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 30000", -- +8 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_sharp"
@@ -109,11 +111,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a broad ax because ...
          descname = _"forging a broad ax",
          actions = {
+            -- time total: 79
             "return=skipped unless economy needs ax_broad",
             "consume=coal:2 iron:2",
-            "sleep=26000",
+            "sleep=38000", -- +12 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 32000", -- +10 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_broad"
@@ -123,11 +126,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a bronze ax because ...
          descname = _"forging a bronze ax",
          actions = {
+            -- time total: 82
             "return=skipped unless economy needs ax_bronze",
             "consume=coal:2 iron:2",
-            "sleep=26000",
+            "sleep=39000", -- +13 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 34000", -- +12 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_bronze"
@@ -137,11 +141,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a battle ax because ...
          descname = _"forging a battle ax",
          actions = {
+            -- time total: 87
             "return=skipped unless economy needs ax_battle",
             "consume=coal gold iron:2",
-            "sleep=26000",
+            "sleep=41000", -- +15 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 37000", -- +15 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_battle"
@@ -151,11 +156,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a warrior’s ax because ...
          descname = _"forging a warrior’s ax",
          actions = {
+            -- time total: 92
             "return=skipped unless economy needs ax_warriors",
             "consume=coal:2 gold:2 iron:2",
-            "sleep=26000",
+            "sleep=43000", -- +17 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 22000",
+            "animate=working 40000", -- +18 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=ax_warriors"

=== modified file 'data/tribes/buildings/productionsites/empire/armorsmithy/init.lua'
--- data/tribes/buildings/productionsites/empire/armorsmithy/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/empire/armorsmithy/init.lua	2016-10-31 07:27:05 +0000
@@ -79,10 +79,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a helmet because ...
          descname = _"forging a helmet",
          actions = {
+            -- time total: 80
             "return=skipped unless economy needs armor_helmet",
-            "sleep=32000",
+            "sleep=40000",  -- +8 enlarge
             "consume=iron coal",
-            "animate=working 35000",
+            "animate=working 40000",  -- +5 enlarge
             "produce=armor_helmet"
          }
       },
@@ -90,10 +91,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a suit of armor because ...
          descname = _"forging a suit of armor",
          actions = {
+            -- time total: 90
             "return=skipped unless economy needs armor",
             "consume=iron coal cloth",
-            "sleep=32000",
-            "animate=working 45000",
+            "sleep=40000",  -- +8 enlarge
+            "animate=working 50000",  -- +5 enlarge
             "produce=armor"
          }
       },
@@ -101,10 +103,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a suit of chain armor because ...
          descname = _"forging a suit of chain armor",
          actions = {
+            -- time total: 108
             "return=skipped unless economy needs armor_chain",
             "consume=iron:2 coal cloth",
-            "sleep=32000",
-            "animate=working 45000",
+            "sleep=47000",  -- +15 enlarge
+            "animate=working 61000",  -- +16 enlarge
             "produce=armor_chain"
          }
       },
@@ -112,10 +115,11 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a suit of gilded armor because ...
          descname = _"forging a suit of gilded armor",
          actions = {
+            -- time total: 115
             "return=skipped unless economy needs armor_gilded",
             "consume=iron:2 coal:2 cloth gold",
-            "sleep=32000",
-            "animate=working 45000",
+            "sleep=51000",  -- +19 enlarge
+            "animate=working 64000",  -- +19 enlarge
             "produce=armor_gilded"
          }
       },

=== modified file 'data/tribes/buildings/productionsites/empire/weaponsmithy/init.lua'
--- data/tribes/buildings/productionsites/empire/weaponsmithy/init.lua	2016-10-24 10:09:01 +0000
+++ data/tribes/buildings/productionsites/empire/weaponsmithy/init.lua	2016-10-31 07:27:05 +0000
@@ -77,11 +77,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a wooden spear because ...
          descname = _"forging a wooden spear",
          actions = {
+            -- time total: 64
             "return=skipped unless economy needs spear_wooden",
-            "sleep=20000",
+            "sleep=27000",  -- +7 enlarge
             "consume=planks",
             "play_sound=sound/smiths smith 192",
-            "animate=working 21000",
+            "animate=working 28000",  -- +7 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=spear_wooden"
@@ -91,11 +92,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a spear because ...
          descname = _"forging a spear",
          actions = {
+            -- time total: 102
             "return=skipped unless economy needs spear",
             "consume=coal iron planks",
-            "sleep=32000",
+            "sleep=47000",  -- +15 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 46000",  -- +10 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=spear"
@@ -105,11 +107,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging an advanced spear because ...
          descname = _"forging an advanced spear",
          actions = {
+            -- time total: 110
             "return=skipped unless economy needs spear_advanced",
             "consume=coal iron:2 planks",
-            "sleep=32000",
+            "sleep=50000",  -- +18 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 51000",  -- +15 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=spear_advanced"
@@ -119,11 +122,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a heavy spear because ...
          descname = _"forging a heavy spear",
          actions = {
+            -- time total: 115
             "return=skipped unless economy needs spear_heavy",
             "consume=coal:2 gold iron planks",
-            "sleep=32000",
+            "sleep=55000",  -- +23 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 51000",  -- +15 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=spear_heavy"
@@ -133,11 +137,12 @@
          -- TRANSLATORS: Completed/Skipped/Did not start forging a war spear because ...
          descname = _"forging a war spear",
          actions = {
+            -- time total: 120
             "return=skipped unless economy needs spear_war",
             "consume=coal:2 gold iron:2 planks",
-            "sleep=32000",
+            "sleep=55000",  -- +23 enlarge
             "play_sound=sound/smiths smith 192",
-            "animate=working 36000",
+            "animate=working 56000",  -- +20 enlarge
             "play_sound=sound/smiths sharpening 120",
             "sleep=9000",
             "produce=spear_war"


Follow ups