Muon regional iterative tracking in default HI reco

[yetkinyilmaz]

Including the muon regional iterative tracking in the default heavy ion reconstruction. From @echapon and @mandrenguyen

[cmsbuild]

A new Pull Request was created by @yetkinyilmaz for CMSSW_7_3_X.

Muon regional iterative tracking in default HI reco

It involves the following packages:

RecoHI/HiMuonAlgos
RecoHI/HiTracking

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@MiheeJo, @jazzitup, @richard-cms, @appeltel, @yenjie, @kurtejung, @mandrenguyen, @dgulhan 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.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[slava77]

Hi Yetkin

Please describe the physics and technical (CPU and such) effects of this PR.
If there were some slides presented in an HI meeting, a link to the slides would be very useful.
This is for self-documentation purpose.

Thank you.

[mandrenguyen]

Hello Slava,

This is the feature presented on slide 4 and 5 from the RECO meeting on
July 31st:
https://indico.cern.ch/event/330432/

Cheers,
Matt

On 11/7/14 9:26 PM, Slava Krutelyov wrote:

Hi Yetkin

Please describe the physics and technical (CPU and such) effects of
this PR.
If there were some slides presented in an HI meeting, a link to the
slides would be very useful.
This is for self-documentation purpose.

Thank you.

—
Reply to this email directly or view it on GitHub
#6279 (comment).

---

Matthew Nguyen
Chargé de Recherche
LLR-Ecole Polytechnique
91128 Palaiseau FRANCE

+33 1 69 33 55 65

[slava77]

please remove, don't comment out

[yetkinyilmaz]

That's a good idea.
@echapon can you add the slides?

On 11/7/14 9:26 PM, Slava Krutelyov wrote:

Hi Yetkin

Please describe the physics and technical (CPU and such) effects of
this PR.
If there were some slides presented in an HI meeting, a link to the
slides would be very useful.
This is for self-documentation purpose.

Thank you.

—
Reply to this email directly or view it on GitHub
#6279 (comment).

[echapon]

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 3a8c59e
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6279/860/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6279/860/summary.html

[slava77]

+1

#6279 3a8c59e
tested in CMSSW_7_3_X_2014-11-07-0200 (test area sign454)

the standalone muons are back.
The rest is the same as already summarized above.

The 140.53 (on 400 events) has a CPU increase of about 3s/event (about 1.5% increase) on average, coming from the hiRegitMu

[cmsbuild]

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.