Use runLog.error and exceptions in getOptimalAssemblyOrientation

armi/physics/fuelCycle/hexAssemblyFuelMgmtUtils.py

@@ -89,9 +89,9 @@ def getOptimalAssemblyOrientation(a: "HexAssembly", aPrev: "HexAssembly") -> int
     """
     maxBuBlock = maxBurnupBlock(a)
     if maxBuBlock.spatialGrid is None:
-        raise ValueError(
-            f"Block {maxBuBlock} in {a} does not have a spatial grid. Cannot rotate."
-        )
+        msg = f"Block {maxBuBlock} in {a} does not have a spatial grid. Cannot rotate."
+        runLog.error(msg)
+        raise ValueError(msg)
     maxBuPinLocation = maxBurnupLocator(maxBuBlock)
     # No need to rotate if max burnup pin is the center
     if maxBuPinLocation.i == 0 and maxBuPinLocation.j == 0:
@@ -105,10 +105,12 @@ def getOptimalAssemblyOrientation(a: "HexAssembly", aPrev: "HexAssembly") -> int
     previousLocations = blockAtPreviousLocation.getPinLocations()
     previousPowers = blockAtPreviousLocation.p.linPowByPin
     if len(previousLocations) != len(previousPowers):
-        raise ValueError(
+        msg = (
             f"Inconsistent pin powers and number of pins in {blockAtPreviousLocation}. "
             f"Found {len(previousLocations)} locations but {len(previousPowers)} powers."
         )
+        runLog.error(msg)
+        raise ValueError(msg)
 
     ringPowers = {
         (loc.i, loc.j): p for loc, p in zip(previousLocations, previousPowers)