remove assert in OuterHitPhiPrediction.h

[VinInn]

remove assert that can fail for straight-line

to fix https://hypernews.cern.ch/HyperNews/CMS/get/recoDevelopment/1488.html
and more

note: this assert has been already removed in 810

[cmsbuild]

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

It involves the following packages:

RecoTracker/TkTrackingRegions

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

cms-bot commands are listed here #13028

[VinInn]

@cmsbuild , please test

[cmsbuild]

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

[davidlange6]

@VinInn - better without "type"..

otherwise, did something change in the last few 80x builds to trigger this problem? (or just bad luck today)

[VinInn]

@davidlange6 not to my knowledge

[VinInn]

wbm says that mag field was 0.294 Tesla in the affected runs

[davidlange6]

ah.. that can help explain it

On Nov 4, 2016, at 9:48 AM, Vincenzo Innocente notifications@github.com wrote:

wbm says that mag field was 0.294 Tesla in the affected runs

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[cmsbuild]

+1
Tested at: 4e1274f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16462/16193/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

[slava77]

+1

for #16462 4e1274f

• assert is removed (commented out, just as in 81X); this affects only events that would otherwise crash
• jenkins tests pass

[cmsbuild]

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

[davidlange6]

+1