fix: Send switch IDs instead of names when initializing.

Author: freezy

VisualPinball.Engine.Mpf.Test/Program.cs

@@ -39,12 +39,17 @@ public static async Task Main(string[] args)
 			var mpfApi = new MpfApi(machineFolder);
 
 			mpfApi.Launch(new MpfConsoleOptions {
-				ShowLogInsteadOfConsole = true,
-				CatchStdOut = true,
+				//ShowLogInsteadOfConsole = true,
+				//CatchStdOut = true,
 			});
 
 			mpfApi.StartGame(new Dictionary<string, bool> {
-				{"sw_11", false},
+				{"1", true},
+				{"2", true},
+				{"3", true},
+				{"4", true},
+				{"5", true},
+				{"6", true},
 			});
 			mpfApi.Client.OnConfigureHardwareRule += (_, request) => {
 				Console.WriteLine($"[MPF] configure hw/rule: sw{request.SwitchNumber} -> c{request.CoilNumber} @{request.HoldPower} | pulse: {request.PulseMs}ms @{request.PulsePower}");

VisualPinball.Engine.Mpf.Unity/Runtime/MpfGamelogicEngine.cs

@@ -81,7 +81,16 @@ public void OnInit(Player player, TableApi tableApi, BallManager ballManager)
 			_api.Client.OnRemoveHardwareRule += OnRemoveHardwareRule;
 			_api.Client.OnFadeLight += OnFadeLight;
 
-			_api.StartGame(player.SwitchStatusesClosed);
+			// map initial switches
+			var mappedSwitchStatuses = new Dictionary<string, bool>();
+			foreach (var swName in player.SwitchStatusesClosed.Keys) {
+				if (_switchIds.ContainsKey(swName)) {
+					mappedSwitchStatuses[_switchIds[swName].ToString()] = player.SwitchStatusesClosed[swName];
+				} else {
+					Logger.Warn($"Unknown intial switch name \"{swName}\".");
+				}
+			}
+			_api.StartGame(mappedSwitchStatuses);
 		}
 
 		public void Switch(string id, bool isClosed)

VisualPinball.Engine.Mpf/MpfClient.cs

@@ -53,7 +53,7 @@ public void StartGame(Dictionary<string, bool> initialSwitches, bool handleStrea
 				ms.InitialSwitchStates.Add(sw, initialSwitches[sw]);
 			}
 
-			Logger.Info("Starting client...");
+			Logger.Info("Starting player with machine state: " + ms);
 			_commandStream = _client.Start(ms);
 
 			if (handleStream) {