Why combining cards two times?

[ram1123]

The combination of cards for each channels and bins are duplicated in two functions in module runHZZFiducialXS.py.

The first function where this is done is createAsimov(), here:

Fiducial_XS/runHZZFiducialXS.py

Lines 394 to 405 in a19f308

	for fState in fStates:
	"""Combine cards for all bins corresponding to each final state.
	"""
	# if (nBins>1):
	cmd = 'combineCards.py '
	for obsBin in range(nBins):
	cmd = cmd + 'hzz4l_'+fState+'S_13TeV_xs_'+obsName.replace(' ','_')+'_bin'+str(obsBin)+'_'+physicalModel+'.txt '
	cmd = cmd + '> hzz4l_'+fState+'S_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt' # Output combine card name
	processCmd(cmd, get_linenumber(), os.path.basename(__file__), 1)
	# combine 3 final state cards
	cmd = 'combineCards.py hzz4l_4muS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt hzz4l_4eS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt hzz4l_2e2muS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt > hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt'

Later, exactly same patch we can find in function extractResults : here

Fiducial_XS/runHZZFiducialXS.py

Lines 591 to 598 in a19f308

	for fState in fStates:
	cmd = 'combineCards.py '
	for bin in range(nBins):
	cmd = cmd+'hzz4l_'+fState+'S_13TeV_xs_'+obsName.replace(' ','_')+'_bin'+str(bin)+'_'+physicalModel+'.txt '
	cmd = cmd + '> hzz4l_'+fState+'S_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt'
	processCmd(cmd, get_linenumber(), os.path.basename(__file__), 1)
	cmd = 'combineCards.py hzz4l_4muS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt hzz4l_4eS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt hzz4l_2e2muS_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt > hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt'

This can be improved.