Wait for scene to load instead of relying on timeout that isn't alway…

…s enough

src/autopack/ips_communication/ips_commands.py

@@ -31,7 +31,7 @@ def create_costfield(ips_instance, harness_setup):
 
 
 def load_scene(ips_instance, scene_file_path):
-    return ips_instance.call("Ips.loadScene", scene_file_path)
+    return ips_instance.call("autopack.loadAndFitScene", scene_file_path)
 
 
 def route_harness_all_solutions(
@@ -119,9 +119,6 @@ def check_distance_of_points(ips_instance, harness_setup, coords, max_geometry_d
 def ergonomic_evaluation(ips_instance, parts, coords):
     ergo_path = pathlib.Path(__file__).parent / "ErgonomicEvaluation.ips"
     load_scene(ips_instance, str(ergo_path.resolve()))
-    import time
-
-    time.sleep(1)
     return ips_instance.call("autopack.evalErgo", parts, coords)
 
 

src/autopack/ips_communication/lualib/autopack.lua

@@ -50,6 +50,17 @@ local function range(from, to)
   return arr
 end
 
+local function loadAndFitScene(scenePath)
+  print("Loading scene " .. scenePath)
+  local loaded = Ips.loadScene(scenePath)
+  -- Fitting the scene helps with two things:
+  -- 1. The scene is loaded in the background, and this makes sure it's
+  --    done before we continue
+  -- 2. It makes it easier to see what's going on
+  Ips.fitScene()
+  return loaded
+end
+
 local function createHarnessRouter(harnessSetup)
   local cableSim = CableSimulation()
   local harnessRouter = HarnessRouter()
@@ -307,6 +318,7 @@ module.ips3VecToTable = ips3VecToTable
 module.ipsNVecToTable = ipsNVecToTable
 module.range = range
 
+module.loadAndFitScene = loadAndFitScene
 module.getCostField = getCostField
 module.setHarnessRouterNodeCosts = setHarnessRouterNodeCosts
 module.routeHarnessSolutions = routeHarnessSolutions