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.

Sign up for GitHub

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

Jump to bottom

add EoP cut for HI #11191

Closed
wants to merge 98 commits into from
Closed

add EoP cut for HI #11191

wants to merge 98 commits into from

Conversation

doanhien
Copy link

@doanhien doanhien commented Sep 8, 2015

set EoP cut on electron in heavy ion reconstruction can reduce significantly fake electrons which
screw up jet energy resolution

dgulhan and others added 30 commits November 24, 2014 15:26
@dgulhan
track analysis update with changes in the track matching
c8108c4
@dgulhan
PileUpSubtractor is modified to take consumesCollector, hiPFJetAnalyz…
ee2bc0a 
…er is updated for track matching, and several other functions changed because of the changes in input they take
@dgulhan
tpRecoAssocGeneralTracks process is added to MC forest configs
17a8213
@dgulhan
53x version without change
b1dd94f
initial commit of photon related code
5d58bde 
Be careful, you now need to make sure that the RechoHI/HiEgammaAlgos folder is present to build properly
@dgulhan
Merge pull request #1 from richard-cms/hiforest_73x_photons
c6b9ee8 
initial commit of photon related code
@dgulhan
53x and 73x versions are combined
0134354
@dgulhan
some classes moved from RecoHI/HiEgammaAlgos
0753a5e
@dgulhan
used in photon pat but missing
d15f079
@dgulhan
these functions are moved to the analysis package
9b872bd
fixes for jets and photons for python imports in progress
ac533d5
@dgulhan
photon tracking collection, efficiency fake src for tracks changed to…
6e54066 
… mergedtruth
@dgulhan
loading the missing HiCoreTools for hiEgammaIsolationSequence
9744004
@dgulhan
forgot to add this for track input change for photons
36151c1
disable tracking efficiency
e304c64
updated centrality provider consumes
cb2c42e
optimistic cleanups of analyzer
6933e80
disable hiEvtAnalyzer, it is fundamentally broken under the new consu…
2fe14ed 
…mes model
photons need serious work, still don't run, isolation parameters are …
7e6fc99 
…not in usual reco
@dgulhan
cutsTPForEff cutsTPForFak are no longer used, changed to MergedTrackT…
6862074 
…ruth
fix photon, ignore pf clusters for now
0efeb6e
specialize CentralityProvider for forest production, needs testing fo…
a3263b4 
…r offline reco
+ggHiNtuplizer of generated particles, photons, electrons
391d2b3
merged ggAnalysis with PhotonAnalysis
5087af1
Merged hiforest_73x from repository dgulhan
e7f9e03 
* dgulhan/hiforest_73x: (21 commits)
  specialize CentralityProvider for forest production, needs testing for offline reco
  fix photon, ignore pf clusters for now
  cutsTPForEff cutsTPForFak are no longer used, changed to MergedTrackTruth
  photons need serious work, still don't run, isolation parameters are not in usual reco
  disable hiEvtAnalyzer, it is fundamentally broken under the new consumes model
  optimistic cleanups of analyzer
  updated centrality provider consumes
  disable tracking efficiency
  forgot to add this for track input change for photons
  loading the missing HiCoreTools for hiEgammaIsolationSequence
  photon tracking collection, efficiency fake src for tracks changed to mergedtruth
  fixes for jets and photons for python imports in progress
  these functions are moved to the analysis package
  used in photon pat but missing
  some classes moved from RecoHI/HiEgammaAlgos
  53x and 73x versions are combined
  initial commit of photon related code
  53x version without change
  tpRecoAssocGeneralTracks process is added to MC forest configs
  PileUpSubtractor is modified to take consumesCollector, hiPFJetAnalyzer is updated for track matching, and several other functions changed because of the changes in input they take
  ...
Merged hiforest_73x_ggAnalysis_latest from repository konush
10cfa02 
* konush/hiforest_73x_ggAnalysis_latest:
  merged ggAnalysis with PhotonAnalysis
  +ggHiNtuplizer of generated particles, photons, electrons
build fix
ca22300
add ggNtuplizer to standard workflow
842a934
added reco muons
2ea9950
Merge pull request #3 from konush/hiforest_ggAnalysis_muons
3b58642 
Added reco muons to ggHiAnalysis
R. Alex Barbieri and others added 20 commits July 16, 2015 16:41
first draft SIM AOD forest config
5ee1a26
@mandrenguyen
Fixes to runForest_PbPb_MIX_75X.py and runForest_PbPb_MIX_75X_bTag.py…
026c0fd 
… broken by recent commits to 750 branch
Merge pull request #15 from appeltel/forest_CMSSW_7_5_0_pre5_ClusterC…
24dc074 
…ompatibility

Enable ClusterCompatibility Filter for AOD
Merge remote-tracking branch 'remotes/cmshi/forest_CMSSW_7_5_0_pre5' …
8cd8137 
…into forest

* remotes/cmshi/forest_CMSSW_7_5_0_pre5:
  enable ClusterCompatibility filter and add collision event selection path for AOD
add AOD collision event selection to HiForest configs
85fcc36
backport for 7_5_X
b9ebf92
backport for 7_5_X
26e2932
backport for 7_5_X
7d9fc26
standalone event analyzer configs
8d39f68
allow disabling the ValuMap Isolation object for old datasets
81a8df4
add special-use-case foresting config for photon commissioning
c0f3611
updates to photon comissioning photon config
98d0a27
forest config for running on old reco
af1da44
disable genparticle tree for photon comissioning, we will use photon …
5d65d6f 
…tree gen info instead
Merged forest_CMSSW_7_5_0 from repository CmsHI
68ce28a
add EoP cut flag for HI
2da6b6f
add EoP cut flag for HI
f748e37
add EoP cut flag for HI
78e334f
add EoP cut flag for HI
e26622b
add EoP cut for HI
66611b3
@cmsbuild cmsbuild added this to the Next CMSSW_7_6_X milestone Sep 8, 2015
@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 8, 2015

A new Pull Request was created by @doanhien for CMSSW_7_6_X.

add EoP cut for HI

It involves the following packages:

HeavyIonsAnalysis/CentralityAnalysis
HeavyIonsAnalysis/Configuration
HeavyIonsAnalysis/EventAnalysis
HeavyIonsAnalysis/JetAnalysis

The following packages do not have a category, yet:

HeavyIonsAnalysis/CentralityAnalysis
HeavyIonsAnalysis/EventAnalysis
HeavyIonsAnalysis/JetAnalysis

@cmsbuild, @vadler, @monttj can you please review it and eventually sign? Thanks.
@istaslis, @mandrenguyen, @yetkinyilmaz this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@doanhien doanhien closed this Sep 8, 2015
@harmonicoscillator harmonicoscillator mentioned this pull request Sep 10, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
analysis-pending comparison-pending new-package-pending orp-pending pending-signatures tests-pending
Projects
None yet
Milestone
CMSSW_7_6_X
Development

Successfully merging this pull request may close these issues.
8 participants
@doanhien @cmsbuild @dgulhan @yetkinyilmaz @mandrenguyen @appeltel @rkunnawa @istaslis