Turn on track selection MVA in Heavy Ions

[istaslis]

Turn on track selection MVA in Heavy Ions after payloads have been uploaded and conditions have been updated.
@mandrenguyen @yetkinyilmaz @VinInn

[cmsbuild]

A new Pull Request was created by @istaslis (Stas Lisniak) for CMSSW_7_5_X.

Turn on track selection MVA in Heavy Ions

It involves the following packages:

RecoHI/HiTracking

@cmsbuild, @cvuosalo, @slava77 can you please review it and eventually sign? Thanks.
@echapon, @jazzitup, @dgulhan, @appeltel, @yenjie, @kurtejung, @mandrenguyen, @richard-cms, @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.

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 519d275
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9391/7979/summary.html

[cvuosalo]

+1

For #9391 519d275

For Heavy Ions, track selection via MVA is enabled.

The code changes are satisfactory. Jenkins tests show no significant differences for all workflows except for RunHI2011wf140p53, as expected. Extended tests for workflows HydjetQ_MinBias_2760GeVwf140p0 and RunHI2011wf140p53 against baseline CMSSW_7_5_X_2015-06-01-110 using 100 events show extensive changes as expected for the change in track selection (see below).
Disk size of reco products changes marginally. The number of tracks increase by about 1-2%, but the resulting size of PF candidates decrease by roughly 5%.
CPU timing appears to improve somewhat. Selector times increase roughly 2-5 times, but amounting to only several ms/event, while PF times are reduced considerably by 5-25%, amounting to 50-400 ms/event (depending on workflow details), and seeming to result in a small net reduction in CPU time.

wf HydjetQ_MinBias_2760GeVwf140p0

wf RunHI2011wf140p53

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1