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

Legacy fiducial and differential xs measurement dev PR #1166

Merged

Conversation

threiten
Copy link
Contributor

@threiten threiten commented Nov 8, 2019

No description provided.

simonepigazzini and others added 30 commits September 24, 2018 10:54
Catalogue for MET checks in ttH analysis
* DiLepton merged into TTHLeptonic

* Met Systematic bug fix
Workspace syncronization (TTHleptonic and THW)
DataFormats/src/classes_def_94X.xml Outdated Show resolved Hide resolved
MetaData/data/cross_sections.json Show resolved Hide resolved
MetaData/python/JobConfig.py Outdated Show resolved Hide resolved
MetaData/work/campaigns/RunIIFall17-3_2_0_part1.json Outdated Show resolved Hide resolved
MetaData/work/campaigns/RunIIFall17-3_2_0_part2.json Outdated Show resolved Hide resolved
Systematics/plugins/PhotonSigEoverEShift.cc Show resolved Hide resolved
Systematics/test/splitTree.py Outdated Show resolved Hide resolved
Taggers/test/MVATraining/checkJsons.py Outdated Show resolved Hide resolved
@simonepigazzini
Copy link
Contributor

@edjtscott, @jonathon-langford , I know you are very busy with the STXS getting to the end of it, but if you can have a look at this running few samples to check nothing breaks on your side it would really help. I think the main sources of troubles could be located in the Dumpers, for the rest the changes are minor (changes in the taggers are usually just indentation or newlines that went in somehow).

@simonepigazzini
Copy link
Contributor

@edjtscott , @jonathon-langford we would really like to merge this, it has been pending since 2016 :). Let me know if you have any objection.

@edjtscott
Copy link
Contributor

sure I will look through the code quickly & start testing now - sorry for the delay

@edjtscott
Copy link
Contributor

FYI I tested again with workspaceStd fixed and all seems to run OK, so once the few remaining changes are made it's fine to merge from my side

@simonepigazzini
Copy link
Contributor

Thanks guys, merging now (finally after ~4 years :) )

@simonepigazzini simonepigazzini merged commit 659617b into cms-analysis:dev_legacy_runII May 26, 2020
sam-may pushed a commit to sam-may/flashgg that referenced this pull request May 29, 2020
…is#1166)

+ Factorized fiducial and differential xs workflow
+ Prompt 2016 analysis strategy
+ Full Run2 optimize categorization

Co-authored-by: Simone Pigazzi <[email protected]>
Co-authored-by: abeschi <[email protected]>
Co-authored-by: Simone Pigazzini <[email protected]>
Co-authored-by: Seth Zenz <[email protected]>
Co-authored-by: vtavolar <[email protected]>
Co-authored-by: Pasquale Musella <[email protected]>
Co-authored-by: gfasanel <[email protected]>
Co-authored-by: Shervin <[email protected]>
Co-authored-by: Nadezda <nadezda.chernyavskaya>
sam-may pushed a commit to sam-may/flashgg that referenced this pull request May 30, 2020
…is#1166)

+ Factorized fiducial and differential xs workflow
+ Prompt 2016 analysis strategy
+ Full Run2 optimize categorization

Co-authored-by: Simone Pigazzi <[email protected]>
Co-authored-by: abeschi <[email protected]>
Co-authored-by: Simone Pigazzini <[email protected]>
Co-authored-by: Seth Zenz <[email protected]>
Co-authored-by: vtavolar <[email protected]>
Co-authored-by: Pasquale Musella <[email protected]>
Co-authored-by: gfasanel <[email protected]>
Co-authored-by: Shervin <[email protected]>
Co-authored-by: Nadezda <nadezda.chernyavskaya>
wamorkart pushed a commit to wamorkart/flashgg that referenced this pull request Nov 19, 2020
…is#1166)

+ Factorized fiducial and differential xs workflow
+ Prompt 2016 analysis strategy
+ Full Run2 optimize categorization

Co-authored-by: Simone Pigazzi <[email protected]>
Co-authored-by: abeschi <[email protected]>
Co-authored-by: Simone Pigazzini <[email protected]>
Co-authored-by: Seth Zenz <[email protected]>
Co-authored-by: vtavolar <[email protected]>
Co-authored-by: Pasquale Musella <[email protected]>
Co-authored-by: gfasanel <[email protected]>
Co-authored-by: Shervin <[email protected]>
Co-authored-by: Nadezda <nadezda.chernyavskaya>
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.

8 participants