Skip to content
Snippets Groups Projects
Commit 747f8ccd authored by Nils Erik Krumnack's avatar Nils Erik Krumnack Committed by Johannes Elmsheuser
Browse files

add an option to add the NOSYS suffix to branches without systematics

add an option to add the NOSYS suffix to branches without systematics

This makes the n-tuple branches more consistent, and makes it very easy
to determine the list of branches for nominal.

It also means that if we add (or drop) systematics from a branch, the
names of the branches in nominal will not change.  That doesn't happen
all that often, but could still be helpful.

Leaving this disabled by default because it (obviously) changes the
n-tuple content.  Also we haven't yet decided whether this is even a
good idea, but it ought to be a good enough idea to allow users to at
least play with it.
parent 6edd7745
No related branches found
No related tags found
30 merge requests!78241Draft: FPGATrackSim: GenScan code refactor,!78236Draft: Switching Streams https://its.cern.ch/jira/browse/ATR-27417,!78056AFP monitoring: new synchronization and cleaning,!78041AFP monitoring: new synchronization and cleaning,!77990Updating TRT chip masks for L1TRT trigger simulation - ATR-28372,!77733Draft: add new HLT NN JVT, augmented with additional tracking information,!77731Draft: Updates to ZDC reconstruction,!77728Draft: updates to ZDC reconstruction,!77522Draft: sTGC Pad Trigger Emulator,!76725ZdcNtuple: Fix cppcheck warning.,!76611L1CaloFEXByteStream: Fix out-of-bounds array accesses.,!76475Punchthrough AF3 implementation in FastG4,!76474Punchthrough AF3 implementation in FastG4,!76343Draft: MooTrackBuilder: Recalibrate NSW hits in refine method,!75729New implementation of ZDC nonlinear FADC correction.,!75703Draft: Update to HI han config for HLT jets,!75184Draft: Update file heavyions_run.config,!74430Draft: Fixing upper bound for Delayed Jet Triggers,!73963Changing the path of the histograms to "Expert" area,!73875updating ID ART reference plots,!73874AtlasCLHEP_RandomGenerators: Fix cppcheck warnings.,!73449Add muon detectors to DarkJetPEBTLA partial event building,!73343Draft: [TrigEgamma] Add photon ringer chains on bootstrap mechanism,!72336Fixed TRT calibration crash,!72176Draft: Improving L1TopoOnline chain that now gets no-empty plots. Activating it by default,!72012Draft: Separate JiveXMLConfig.py into Config files,!71876Fix MET trigger name in MissingETMonitoring,!71820Draft: Adding new TLA End-Of-Fill (EOF) chains and removing obsolete DIPZ chains,!71279Draft: ATR-29330: Move L1_4J15 and the HLT chains seeded by it in the MC Menu,!71193add an option to add the NOSYS suffix to branches without systematics
......@@ -47,6 +47,9 @@ class OutputAnalysisConfig (ConfigBlock):
"prefaced by the keywords enable or disable) to turn on/off the "
"writing of branches to the output ntuple. The default is None "
"(no modification to the scheduled output branches).")
self.addOption ('alwaysAddNosys', False, type=bool,
info="If set to True, all branches will be given a systematics suffix, "
"even if they have no systematics (beyond the nominal).")
def makeAlgs (self, config) :
......@@ -114,6 +117,8 @@ class OutputAnalysisConfig (ConfigBlock):
if outputConfig.noSys :
outputConfig.outputContainerName = outputConfig.outputContainerName.replace ('%SYS%', 'NOSYS')
outputConfig.variableName = outputConfig.variableName.replace ('%SYS%', 'NOSYS')
if self.alwaysAddNosys :
outputName += "_NOSYS"
else :
outputName += '_%SYS%'
myVars += [outputConfig.outputContainerName + '.' + outputConfig.variableName + ' -> ' + outputName]
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment