rules-based terrain cost, part 1

OpenRA.FileFormats/Session.cs

@@ -61,7 +61,7 @@ namespace OpenRA.FileFormats
 		public readonly string[] 
 			Folders, Packages, LegacyRules, Rules, 
 			Sequences, Chrome, Assemblies, ChromeLayout, 
-			Weapons, Voices;
+			Weapons, Voices, Terrain;
 
 		public Manifest(string[] mods)
 		{
@@ -79,6 +79,7 @@ namespace OpenRA.FileFormats
 			ChromeLayout = YamlList(yaml, "ChromeLayout");
 			Weapons = YamlList(yaml, "Weapons");
 			Voices = YamlList(yaml, "Voices");
+			Terrain = YamlList(yaml, "Terrain");
 		}
 
 		static string[] YamlList(Dictionary<string, MiniYaml> ys, string key) { return ys[key].Nodes.Keys.ToArray(); }

OpenRA.Game/GameRules/Rules.cs

@@ -34,6 +34,7 @@ namespace OpenRA
 		public static Dictionary<string, ActorInfo> Info;
 		public static Dictionary<string, WeaponInfo> Weapons;
 		public static Dictionary<string, VoiceInfo> Voices;
+		public static Dictionary<TerrainType, TerrainCost> TerrainTypes;
 
 		public static void LoadRules(string map, Manifest m)
 		{
@@ -48,6 +49,8 @@ namespace OpenRA
 			Info = LoadYamlRules(m.Rules, (k, y) => new ActorInfo(k.Key.ToLowerInvariant(), k.Value, y));
 			Weapons = LoadYamlRules(m.Weapons, (k, _) => new WeaponInfo(k.Key.ToLowerInvariant(), k.Value));
 			Voices = LoadYamlRules(m.Voices, (k, _) => new VoiceInfo(k.Value));
+			TerrainTypes = LoadYamlRules(m.Terrain, (k, _) => new TerrainCost(k.Value))
+				.ToDictionary(kv => (TerrainType)Enum.Parse(typeof(TerrainType), kv.Key, true), kv => kv.Value);
 
 			TechTree = new TechTree();
 		}

OpenRA.Game/GameRules/TerrainCost.cs

@@ -18,10 +18,11 @@
  */
 #endregion
 
-using OpenRA.Graphics;
+using System;
 using OpenRA.FileFormats;
+using OpenRA.Graphics;
 
-namespace OpenRA
+namespace OpenRA.GameRules
 {
 	public enum UnitMovementType : byte
 	{
@@ -32,35 +33,26 @@ namespace OpenRA
 		Fly = 4,
 	}
 
-	static class TerrainCosts
+	public class TerrainCost
 	{
-		static float[][] costs = Util.MakeArray<float[]>(4,
+		public readonly bool Buildable = true;
-			a => Util.MakeArray<float>(11, b => float.PositiveInfinity));
+		public readonly float Foot = 0, Track = 0, Wheel = 0, Float = 0;
-		
+		public readonly bool AcceptSmudge = true;
-		static bool[] buildable = Util.MakeArray<bool>(11,b => false);
+
-		static TerrainCosts()
+		public TerrainCost(MiniYaml y) { FieldLoader.Load(this, y); }
+
+		public float GetCost(UnitMovementType umt)
 		{
-			for( int i = 0 ; i < 11 ; i++ )
+			switch (umt)			/* todo: make this nice */
 			{
-				if( i == 4 ) continue;
+				case UnitMovementType.Fly: return 1;
-				var section = Rules.AllRules.GetSection( ( (TerrainType)i ).ToString() );
+				case UnitMovementType.Foot: return 1 / Foot;
-				for( int j = 0 ; j < 4 ; j++ )
+				case UnitMovementType.Wheel: return 1 / Wheel;
-				{
+				case UnitMovementType.Track: return 1 / Track;
-					string val = section.GetValue( ( (UnitMovementType)j ).ToString(), "0%" );
+				case UnitMovementType.Float: return 1 / Float;
-					costs[j][i] = 100f / float.Parse(val.Substring(0, val.Length - 1));
+				default:
-				}
+					throw new InvalidOperationException("wtf?");
-				buildable[i] = (section.GetValue("Buildable", "no") == "yes");
 			}
 		}
-		
-		public static bool Buildable(TerrainType r)
-		{
-			return buildable[(int)r];
-		}
-		
-		public static float Cost( UnitMovementType unitMovementType, TerrainType r )
-		{
-			return costs[ (byte)unitMovementType ][ (int)r ];
-		}
 	}
 }

OpenRA.Game/OpenRA.Game.csproj

@@ -188,7 +188,7 @@
     <Compile Include="Graphics\Sprite.cs" />
     <Compile Include="Graphics\SpriteRenderer.cs" />
     <Compile Include="Graphics\SpriteSheetBuilder.cs" />
-    <Compile Include="TerrainCosts.cs" />
+    <Compile Include="GameRules\TerrainCost.cs" />
     <Compile Include="Graphics\TerrainRenderer.cs" />
     <Compile Include="Traits\Activities\Harvest.cs" />
     <Compile Include="Traits\Activities\Move.cs" />

OpenRA.Game/PathFinder.cs

@@ -24,6 +24,7 @@ using System.Diagnostics;
 using System.Linq;
 using OpenRA.Support;
 using OpenRA.Traits;
+using OpenRA.GameRules;
 
 namespace OpenRA
 {
@@ -38,11 +39,12 @@ namespace OpenRA
 			var map = world.Map;
 			for (var umt = UnitMovementType.Foot; umt <= UnitMovementType.Float; umt++)
 				passableCost[(int)umt] = new float[map.MapSize, map.MapSize];
-			for( int x = 0 ; x < map.MapSize ; x++ )
+			for (int x = 0; x < map.MapSize; x++)
-				for( int y = 0 ; y < map.MapSize ; y++ )
+				for (int y = 0; y < map.MapSize; y++)
-					for (var umt = UnitMovementType.Foot; umt <= UnitMovementType.Float; umt++ )
+					for (var umt = UnitMovementType.Foot; umt <= UnitMovementType.Float; umt++)
-						passableCost[(int)umt][ x, y ] = ( world.Map.IsInMap( x, y ) )
+						passableCost[(int)umt][x, y] = (world.Map.IsInMap(x, y))
-							? (float)TerrainCosts.Cost( umt, world.TileSet.GetTerrainType( world.Map.MapTiles[ x, y ] ) )
+							? (float)Rules.TerrainTypes[world.TileSet.GetTerrainType(world.Map.MapTiles[x, y])]
+								.GetCost(umt)
 							: float.PositiveInfinity;
 		}
 

OpenRA.Game/PathSearch.cs

@@ -23,6 +23,7 @@ using System.Collections.Generic;
 using System.Linq;
 using OpenRA.FileFormats;
 using OpenRA.Traits;
+using OpenRA.GameRules;
 
 namespace OpenRA
 {

OpenRA.Game/Traits/Activities/Harvest.cs

@@ -19,6 +19,7 @@
 #endregion
 
 using System.Linq;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits.Activities
 {

OpenRA.Game/Traits/Activities/HeliLand.cs

@@ -18,6 +18,7 @@
  */
 #endregion
 
+using OpenRA.GameRules;
 namespace OpenRA.Traits.Activities
 {
 	class HeliLand : IActivity

OpenRA.Game/Traits/Activities/UnloadCargo.cs

@@ -19,6 +19,7 @@
 #endregion
 
 using System.Linq;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits.Activities
 {

OpenRA.Game/Traits/Bridge.cs

@@ -23,6 +23,7 @@ using System.Drawing;
 using System.Linq;
 using OpenRA.FileFormats;
 using OpenRA.Graphics;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {
@@ -128,10 +129,7 @@ namespace OpenRA.Traits
 
 		public float GetCost(int2 p, UnitMovementType umt)
 		{
-			// just use the standard walkability from templates.ini. no hackery.
+			return Rules.TerrainTypes[Templates[state].TerrainType[Tiles[p]]].GetCost(umt);
-
-			return TerrainCosts.Cost(umt, 
-				Templates[state].TerrainType[Tiles[p]]);
 		}
 
 		static bool IsIntact(Bridge b)

OpenRA.Game/Traits/Cargo.cs

@@ -20,6 +20,7 @@
 
 using System.Collections.Generic;
 using OpenRA.Traits.Activities;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {

OpenRA.Game/Traits/ConstructionYard.cs

@@ -19,6 +19,7 @@
 #endregion
 
 using OpenRA.Traits.Activities;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {
@@ -66,10 +67,7 @@ namespace OpenRA.Traits
 		}
 	
 		// HACK: This should make reference to an MCV actor, and use of its Mobile trait
-		public UnitMovementType GetMovementType()
+		public UnitMovementType GetMovementType() { return UnitMovementType.Wheel; }
-		{
-			return UnitMovementType.Wheel;
-		}
 		
 		public bool CanEnterCell(int2 a)
 		{
@@ -88,10 +86,10 @@ namespace OpenRA.Traits
 			}
 			
 			if (!crushable) return false;
-			
+
 			return self.World.Map.IsInMap(a.X, a.Y) &&
-				TerrainCosts.Cost(GetMovementType(),
+				Rules.TerrainTypes[self.World.TileSet.GetTerrainType(self.World.Map.MapTiles[a.X, a.Y])]
-					self.World.TileSet.GetTerrainType(self.World.Map.MapTiles[a.X, a.Y])) < double.PositiveInfinity;
+				.GetCost(GetMovementType()) < float.PositiveInfinity;
 		}
 	}
 }

OpenRA.Game/Traits/Crate.cs

@@ -21,6 +21,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using OpenRA.FileFormats;
+using OpenRA.GameRules;
 
 /*
  * Crates left to implement:
@@ -75,24 +76,15 @@ namespace OpenRA.Traits
 					n -= s.Second;
 		}
 
-		public bool IsPathableCrush(UnitMovementType umt, Player player)
+		public bool IsPathableCrush(UnitMovementType umt, Player player) { return true; }
-		{
+		public bool IsCrushableBy(UnitMovementType umt, Player player) { return true; }
-			return true;
-		}
-
-		public bool IsCrushableBy(UnitMovementType umt, Player player)
-		{
-			return true;
-		}
 
 		public IEnumerable<int2> OccupiedCells() { yield return self.Location; }
 		
 		public void Tick(Actor self)
 		{
 			if (++ticks >= self.Info.Traits.Get<CrateInfo>().Lifetime * 25)
-			{
 				self.World.AddFrameEndTask(w =>	w.Remove(self));
-			}
 		}
 	}
 }

OpenRA.Game/Traits/Helicopter.cs

@@ -20,6 +20,7 @@
 
 using System;
 using OpenRA.Traits.Activities;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {

OpenRA.Game/Traits/Mobile.cs

@@ -20,6 +20,7 @@
 
 using System.Collections.Generic;
 using System.Linq;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {
@@ -131,10 +132,10 @@ namespace OpenRA.Traits
 			}
 			
 			if (!crushable) return false;
-			
+
 			return self.World.Map.IsInMap(a.X, a.Y) &&
-				TerrainCosts.Cost(GetMovementType(),
+				Rules.TerrainTypes[self.World.TileSet.GetTerrainType(self.World.Map.MapTiles[a.X, a.Y])]
-					self.World.TileSet.GetTerrainType(self.World.Map.MapTiles[a.X, a.Y])) < double.PositiveInfinity;
+				.GetCost(GetMovementType()) < float.PositiveInfinity;
 		}
 
 		public IEnumerable<int2> GetCurrentPath()

OpenRA.Game/Traits/Plane.cs

@@ -21,6 +21,7 @@
 using System;
 using System.Linq;
 using OpenRA.Traits.Activities;
+using OpenRA.GameRules;
 
 namespace OpenRA.Traits
 {

OpenRA.Game/Traits/SquishByTank.cs

@@ -18,6 +18,7 @@
  */
 #endregion
 
+using OpenRA.GameRules;
 namespace OpenRA.Traits
 {
 	class SquishByTankInfo : ITraitInfo

OpenRA.Game/WorldUtils.cs

@@ -34,8 +34,10 @@ namespace OpenRA
 		{
 			if (world.WorldActor.traits.Get<BuildingInfluence>().GetBuildingAt(a) != null) return false;
 			if (world.WorldActor.traits.Get<UnitInfluence>().GetUnitsAt(a).Any()) return false;
-			
+
-			return world.Map.IsInMap(a.X, a.Y) && TerrainCosts.Cost(umt, world.TileSet.GetTerrainType(world.Map.MapTiles[a.X,a.Y])) < double.PositiveInfinity;
+			return world.Map.IsInMap(a.X, a.Y) &&
+				Rules.TerrainTypes[world.TileSet.GetTerrainType(world.Map.MapTiles[a.X, a.Y])]
+				.GetCost(umt) < float.PositiveInfinity;
 		}
 		
 		public static bool IsCellBuildable(this World world, int2 a, bool waterBound)
@@ -50,8 +52,10 @@ namespace OpenRA
 			
 			if (waterBound)
 				return world.Map.IsInMap(a.X,a.Y) && GetTerrainType(world,a) == TerrainType.Water;
-			
+
-			return world.Map.IsInMap(a.X, a.Y) && TerrainCosts.Buildable(world.TileSet.GetTerrainType(world.Map.MapTiles[a.X, a.Y]));
+			return world.Map.IsInMap(a.X, a.Y) &&
+				Rules.TerrainTypes[world.TileSet.GetTerrainType(world.Map.MapTiles[a.X, a.Y])]
+				.Buildable;
 		}
 
 		public static bool IsActorCrushableByActor(this World world, Actor a, Actor b)

OpenRA.Mods.RA/Mine.cs

@@ -23,6 +23,7 @@ using System.Linq;
 using OpenRA.Effects;
 using OpenRA.Traits;
 using OpenRA.Traits.Activities;
+using OpenRA.GameRules;
 
 namespace OpenRA.Mods.RA
 {

mods/cnc/minimal.ini

@@ -14,86 +14,3 @@ LowPowerSlowdown=3		; slowdown factor for low power
 
 GapRegenInterval=.1     ; gap generators will regenerate their shroud at this time interval
 SubmergeDelay=.02       ; forced delay that subs will remain on surface before allowing to submerge
-
-
-; ******* Land Characteristics *******
-
-; clear grassy terrain
-[Clear]
-Foot=90%
-Track=80%
-Wheel=60%
-Float=0%
-Buildable=yes
-
-; rocky terrain
-[Rough]
-Foot=80%
-Track=70%
-Wheel=40%
-Float=0%
-Buildable=no
-
-; roads
-[Road]
-Foot=100%
-Track=100%
-Wheel=100%
-Float=0%
-Buildable=yes
-
-; open water
-[Water]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=100%
-Buildable=no
-
-; cliffs
-[Rock]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; walls and other man made obstacles
-[Wall]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; ore (Tiberium)
-[Ore]
-Foot=90%
-Track=70%
-Wheel=50%
-Float=0%
-Buildable=no
-
-; sandy beach
-[Beach]
-Foot=80%
-Track=70%
-Wheel=40%
-Float=0%
-Buildable=no
-
-; craggy riverbed
-[River]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; special tile for bridge hacks
-[Special]
-Foot=100%
-Track=100%
-Wheel=100%
-Float=100%
-Buildable=no

mods/cnc/mod.yaml

@@ -50,4 +50,7 @@ Weapons:
 	mods/cnc/weapons.yaml:
 	
 Voices:
-	mods/cnc/voices.yaml:
+	mods/cnc/voices.yaml:
+	
+Terrain:
+	mods/cnc/terrain.yaml

mods/cnc/terrain.yaml

@@ -0,0 +1,82 @@
+Clear:
+	Foot: 90%
+	Track: 80%
+	Wheel: 60%
+	Float: 0%
+	Buildable: yes
+
+Rough:
+	Foot: 80%
+	Track: 70%
+	Wheel: 40%
+	Float: 0%
+	Buildable: no
+
+Road:
+	Foot: 100%
+	Track: 100%
+	Wheel: 100%
+	Float: 0%
+	Buildable: yes
+	
+Tree:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Water:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 100%
+	Buildable: no
+	AcceptSmudge: no
+
+Rock:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Wall:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+
+Ore:
+	Foot: 90%
+	Track: 70%
+	Wheel: 50%
+	Float: 0%
+	Buildable: no
+
+Beach:
+	Foot: 80%
+	Track: 70%
+	Wheel: 40%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+River:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Special:
+	Foot: 100%
+	Track: 100%
+	Wheel: 100%
+	Float: 100%
+	Buildable: no
+	AcceptSmudge: no

mods/ra/mod.yaml

@@ -43,4 +43,7 @@ Weapons:
 	mods/ra/weapons.yaml
 	
 Voices:
-	mods/ra/voices.yaml
+	mods/ra/voices.yaml
+	
+Terrain:
+	mods/ra/terrain.yaml

mods/ra/rules.ini

@@ -88,94 +88,3 @@ TeamDelay=.6            ; interval between checking for and creating teams
 ; misc
 FineDiffControl=no      ; Allow 5 difficulty settings instead of only 3 settings?
 LowPowerSlowdown=3
-
-; ******* Land Characteristics *******
-; This section specifies the characteristics of the various
-; terrain types. The primary purpose is to differentiate the
-; movement capabilities.
-
-; Float = % of full speed for ships [0 means impassable] (def=100)
-; Foot = % of full speed for foot soldiers [0 means impassable] (def=100)
-; Track = % of full speed for tracked vehicles [0 means impassable] (def=100)
-; Wheel = % of full speed for wheeled vehicles [0 means impassable] (def=100)
-; Buildable = Can buildings be built upon this terrain (def=no)?
-
-; clear grassy terrain
-[Clear]
-Foot=90%
-Track=80%
-Wheel=60%
-Float=0%
-Buildable=yes
-
-; rocky terrain
-[Rough]
-Foot=80%
-Track=70%
-Wheel=40%
-Float=0%
-Buildable=no
-
-; roads
-[Road]
-Foot=100%
-Track=100%
-Wheel=100%
-Float=0%
-Buildable=yes
-
-; open water
-[Water]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=100%
-Buildable=no
-
-; cliffs
-[Rock]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; walls and other man made obstacles
-[Wall]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; ore (Tiberium)
-[Ore]
-Foot=90%
-Track=70%
-Wheel=50%
-Float=0%
-Buildable=no
-
-; sandy beach
-[Beach]
-Foot=80%
-Track=70%
-Wheel=40%
-Float=0%
-Buildable=no
-
-; craggy riverbed
-[River]
-Foot=0%
-Track=0%
-Wheel=0%
-Float=0%
-Buildable=no
-
-; special tile for bridge hacks
-[Special]
-Foot=100%
-Track=100%
-Wheel=100%
-Float=100%
-Buildable=no

mods/ra/terrain.yaml

@@ -0,0 +1,82 @@
+Clear:
+	Foot: 90%
+	Track: 80%
+	Wheel: 60%
+	Float: 0%
+	Buildable: yes
+
+Rough:
+	Foot: 80%
+	Track: 70%
+	Wheel: 40%
+	Float: 0%
+	Buildable: no
+
+Road:
+	Foot: 100%
+	Track: 100%
+	Wheel: 100%
+	Float: 0%
+	Buildable: yes
+	
+Tree:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Water:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 100%
+	Buildable: no
+	AcceptSmudge: no
+
+Rock:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Wall:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+
+Ore:
+	Foot: 90%
+	Track: 70%
+	Wheel: 50%
+	Float: 0%
+	Buildable: no
+
+Beach:
+	Foot: 80%
+	Track: 70%
+	Wheel: 40%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+River:
+	Foot: 0%
+	Track: 0%
+	Wheel: 0%
+	Float: 0%
+	Buildable: no
+	AcceptSmudge: no
+
+Special:
+	Foot: 100%
+	Track: 100%
+	Wheel: 100%
+	Float: 100%
+	Buildable: no
+	AcceptSmudge: no