Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Signal Parametrisation & JES obs fix #65

Open
wants to merge 82 commits into
base: CMSSW_10_X
Choose a base branch
from

Conversation

ram1123
Copy link
Collaborator

@ram1123 ram1123 commented Jul 24, 2022

No description provided.

ram1123 and others added 30 commits May 4, 2022 13:37
- Added additional append lines for the command output txt files
- Added missing import of GetDirectory in the plotDifferentialBins.py
* Added year in the combine output root files

* moved all combine outputs to the combineOutputs directory
- Still need to add the plotting functionalities for the combined year.
- Updated few instances of `if (obsName.startswith("mass4l")):` to if (obsName == "mass4l" ):
- Updated plotDifferentialBins.py script as per https://github.com/tjavaid/Fiducial_XS_combination/blob/main/plotDifferentialBins.py
- Added year to the instance where template XS was running related to #49
- Added JES info to datacard. Just reading the JES dict created using `read_jes` module and appending info at the end of datacard. Related to #49
- Updated year positon in the template maker function `fiducialXSTemplates`
@@ -597,6 +642,9 @@ def createAsimov(obsName, observableBins, modelName, resultsXS, physicalModel, y
if (physicalModel=="v2"):
cmd = 'text2workspace.py hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt -P HiggsAnalysis.CombinedLimit.HZZ4L_Fiducial_v2:differentialFiducialV2 --PO higgsMassRange=115,135 --PO nBin='+str(nBins)+' -o hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.root'
processCmd(cmd, get_linenumber(), os.path.basename(__file__))
if (physicalModel=="v3"):
cmd = 'text2workspace.py hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt -P HiggsAnalysis.CombinedLimit.HZZ4L_Fiducial:differentialFiducialV3 --PO higgsMassRange=115,135 --PO nBin='+str(nBins)+' -o hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.root'
processCmd(cmd, get_linenumber(), os.path.basename(__file__))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this + comment here?

@@ -597,6 +642,9 @@ def createAsimov(obsName, observableBins, modelName, resultsXS, physicalModel, y
if (physicalModel=="v2"):
cmd = 'text2workspace.py hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt -P HiggsAnalysis.CombinedLimit.HZZ4L_Fiducial_v2:differentialFiducialV2 --PO higgsMassRange=115,135 --PO nBin='+str(nBins)+' -o hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.root'
processCmd(cmd, get_linenumber(), os.path.basename(__file__))
if (physicalModel=="v3"):
cmd = 'text2workspace.py hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.txt -P HiggsAnalysis.CombinedLimit.HZZ4L_Fiducial:differentialFiducialV3 --PO higgsMassRange=115,135 --PO nBin='+str(nBins)+' -o hzz4l_all_13TeV_xs_'+obsName.replace(' ','_')+'_bin_'+physicalModel+'.root'
processCmd(cmd, get_linenumber(), os.path.basename(__file__))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants