fix det search tolerances for Cosmics

[VinInn]

the title says all

[VinInn]

@cmsbuild , please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/12930/console

[cmsbuild]

A new Pull Request was created by @VinInn (Vincenzo Innocente) for CMSSW_8_1_X.

It involves the following packages:

RecoTracker/CkfPattern
TrackingTools/DetLayers

@cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @bellan, @mschrode, @gpetruc, @dgulhan this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[cmsbuild]

+1
Tested at: 750f907
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14466/12930/summary.html

[cmsbuild]

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

[davidlange6]

@VinInn - just to clarify - I had understood the missing rate issue was limited to B=0 - does this PR also have a significant effect at B=3.8?

[VinInn]

We do not know the effect at 3.8T.
In any case it is consistent with what used elsewhere for instance for Muon seeded, conversion, electron etc
I asked Matti to use it also for LowPU.
Essentially makes it consistent with RunI and 7X

[slava77]

+1

for #14466 750f907

• configuration change is in line with the needed fix; the change in GroupedCkfTrajectoryBuilderP5_cff.py is the same as in fix det search tollerances for Cosmics #14467 0c85e7e; there is also an addition of a few LogDebugs here (OK)
• jenkins tests pass and comparisons with baseline show small differences in the cosmics processing workflow 4.22 and also in cosmics-related variables (only in 25202, probably just a rare value to change)
• see notes on local tests in fix det search tollerances for Cosmics #14467 (comment)

[cmsbuild]

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

[davidlange6]

+1