From 299dac1e3a6e1966de75df2b0be078da97eb952b Mon Sep 17 00:00:00 2001
From: Edward Moyse <Edward.Moyse@cern.ch>
Date: Sun, 10 Jul 2016 06:30:34 +0200
Subject: [PATCH] Set runnumber correctly for bunch spacing autoconfiguration
 (RecExConfig-00-03-68)

	* Whoops - set runnumber correctly for bunch spacing
	* tag RecExConfig-00-03-68

2016-07-06 Edward.Moyse@cern.ch
	* For run-1, set bunch spacing to 50ns (which is default, but we'll do it explicitly for clarity, and in case we change the default)
	* tag RecExConfig-00-03-67

2016-07-06 Edward.Moyse@cern.ch
	* If no data, set autoconfigure to 25ns (as discussed in ATLASRECTS-3197)
	* tag RecExConfig-00-03-66
---
 .../RecExConfig/python/AutoConfiguration.py   | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/Reconstruction/RecExample/RecExConfig/python/AutoConfiguration.py b/Reconstruction/RecExample/RecExConfig/python/AutoConfiguration.py
index 4267eba21b9..1ad5aba7cf8 100644
--- a/Reconstruction/RecExample/RecExConfig/python/AutoConfiguration.py
+++ b/Reconstruction/RecExample/RecExConfig/python/AutoConfiguration.py
@@ -481,15 +481,28 @@ def ConfigureBeamBunchSpacing():
         if not jobproperties.Beam.beamType.is_locked():
             ConfigureBeamType()
         if jobproperties.Beam.beamType() == 'collisions':
+            runnumber = GetRunNumber()
+            if runnumber is None:
+                  logAutoConfiguration.info("Couldn't get run number. Aborting.")
+                  return
+            if (runnumber<236107): # run-1
+                  logAutoConfiguration.info("Run-1 data, set to 50ns.")
+                  jobproperties.Beam.bunchSpacing.set_Value_and_Lock(50)
+                  # could have just left default, but at least this is explicit.
+                  pass
             #Real data collision run, try to get bunch spacing from database
             from CoolConvUtilities.BunchSpacingUtils import bunchSpacingOfRun
-            bc=bunchSpacingOfRun(GetRunNumber(),GetLBNumber())
+            bc=bunchSpacingOfRun(runnumber,GetLBNumber())
             if bc is not None and bc==1: 
                 jobproperties.Beam.bunchSpacing.set_Value_and_Lock(25)
                 logAutoConfiguration.info("Autoconfigure bunch-spacing to 25 ns")
             else:
-                logAutoConfiguration.info("No data or larger bunch spacing, autoconfigure to 50 ns")
-                pass
+                if bc is None:
+                  logAutoConfiguration.info("No data, autoconfigure to 25 ns")
+                  jobproperties.Beam.bunchSpacing.set_Value_and_Lock(25)
+                else: 
+                  logAutoConfiguration.info("No data or larger bunch spacing, autoconfigure to 50 ns")
+                  pass
             pass
         else:
             logAutoConfiguration.info("Not a collisions run, bunch spacing not autoconfigured")
-- 
GitLab