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

Including DYT algorithm into TuneP and PFMuon #6014

Merged
merged 11 commits into from Oct 29, 2014

Conversation

trocino
Copy link
Contributor

@trocino trocino commented Oct 27, 2014

Hi,
this PR is to insert the DYnamic Truncation (DYT) muon refit into the TuneP algorithm, which chooses the best muon track among different refits (originally tracker, global, TPFMS, Picky, now also DYT), according to the track quality. (@rradogna, @ptraczyk)
The output of TuneP (other than being available in reco::Muon as "tunePMuonBestTrack") is passed to the PFMuonAlgo, which can modify the choice by TuneP and select a different track or discard the muon, based on the whole event reconstruction, in particular the effect of the muon on the PF MET. The PFMuonAlgo choice is available in reco::Muon as "muonBestTrack". So also PFMuonAlgo (and PFCandidate) needed some adjustments to handle the new refit type.
In the attached plot by @rradogna, you can see a comparison between the performance of the original TuneP (blue) and the new TuneP with DYT (red) in reconstructing the pt of muons from a SingleMuPt1000 gun sample. The green distribution shows the selected pt between Picky or DYT, when available. It can be seen that the performances of the old and new TuneP are similar, but the new one has less tails (compare RMS, overflow). Note that both DYT and Picky will be further optimized later.
More performance plots from a Zprime(2250) sample can be found here:
http://trocino.web.cern.ch/trocino/Temp/2014-10-27_NewTuneP/
Blue plots are for the old TuneP/PFMuonAlgo, red plots are for the new version. You can see the pt resolution of "tunePMuonBestTrack" (from TuneP) and "muonBestTrack" (from PF) for all muons with a valid inner track ("AllMuons_"), for muons passing the HighPt selection ("HighPtMuons_"), and for muons passing the PFMuon selection ("PFMuons_*"). In all cases, you can see an improvement in the resolution, especially the tails. There's also a PFMET distribution, which looks significantly improved.
@abbiendi, @battibass, @bachtis, @bellan
tunep_performances

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @trocino (Daniele Trocino) for CMSSW_7_3_X.

Including DYT algorithm into TuneP and PFMuon

It involves the following packages:

DataFormats/MuonReco
DataFormats/ParticleFlowCandidate
RecoMuon/MuonIdentification
RecoParticleFlow/PFProducer

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@mmarionncern, @battibass, @abbiendi, @lgray, @bellan, @bachtis, @rociovilar 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.

tevTk = muon::tevOptimized(*muon, *trackMapDefault, *trackMapFirstHit,
*trackMapPicky);
tevTk = muon::tevOptimized(*muon);
//tevTk = muon::tevOptimized(*muon, *trackMapDefault, *trackMapFirstHit,
Copy link
Contributor

Choose a reason for hiding this comment

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

is this commented out part needed for imminent development?
If not, please, remove

Copy link
Contributor

Choose a reason for hiding this comment

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

This commented line is not needed, we will remove it.

@trocino
Copy link
Contributor Author

trocino commented Oct 27, 2014

Implemented Slava's and Stoyan's comments (thanks!), and added pt>0 checks in few other points in the same code.

@StoyanStoynev
Copy link
Contributor

@cmsbuild

@cmsbuild
Copy link
Contributor

Pull request #6014 was updated. @cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please check and sign again.

@StoyanStoynev
Copy link
Contributor

@cmsbuild please test

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@StoyanStoynev
Copy link
Contributor

+1
For 2f337f5 .
I tested a previous tag of this PR and showed results above. I compared the histograms from the previous tag with the one I am signing off and found no diffs (both on top of CMSSW_7_3_X_2014-10-25-1400; short and extended matrix tests; FWLite and DQM based comparisons). Thus conclusions remain - no regressions, statistics is not enough to confirm improvement in tails, the new DYT algo is seen to be properly included in TuneP.

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request will be automatically merged.

cmsbuild added a commit that referenced this pull request Oct 29, 2014
Including DYT algorithm into TuneP and PFMuon
@cmsbuild cmsbuild merged commit 35532c5 into cms-sw:CMSSW_7_3_X Oct 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants