Add sanity checks for initial/final bunch crossing arguments

Simulation/Digitization/python/DigitizationConfigFlags.py

@@ -224,6 +224,10 @@ def pileupRunArgsToFlags(runArgs, flags):
     if hasattr(runArgs, "pileupFinalBunch"):
         flags.Digitization.PU.FinalBunchCrossing = runArgs.pileupFinalBunch
 
+    # sanity check
+    if flags.Digitization.PU.InitialBunchCrossing > flags.Digitization.PU.FinalBunchCrossing:
+        raise ValueError("Initial bunch crossing should not be larger than the final one")
+
     if hasattr(runArgs, "inputLowPtMinbiasHitsFile"):
         from Digitization.PileUpUtils import generateBackgroundInputCollections
         flags.Digitization.PU.LowPtMinBiasInputCols = \

Simulation/SimuJobTransforms/share/skeleton.HITtoRDO.py

@@ -121,6 +121,8 @@ if hasattr(runArgs,"digiSteeringConf"):
         digilog.info( "Changing digitizationFlags.digiSteeringConf from %s to %s", digitizationFlags.digiSteeringConf.get_Value(),runArgs.digiSteeringConf)
         digitizationFlags.digiSteeringConf=runArgs.digiSteeringConf+"PileUpToolsAlg"
         PileUpConfigOverride=True
+if digitizationFlags.initialBunchCrossing > digitizationFlags.finalBunchCrossing:
+    raise ValueError( "Initial bunch crossing should not be larger than the final one" )
 if PileUpConfigOverride:
     digilog.info( "NB Some pile-up (re-)configuration was done on the command-line.")
 del PileUpConfigOverride