New track algorithm names for HI muon RegIt

[echapon]

75X equivalent of #10653

[cmsbuild]

A new Pull Request was created by @echapon (Emilien Chapon) for CMSSW_7_5_X.

New track algorithm names for HI muon RegIt

It involves the following packages:

DataFormats/TrackReco
RecoHI/HiMuonAlgos

@cmsbuild, @cvuosalo, @slava77 can you please review it and eventually sign? Thanks.
@makortel, @MiheeJo, @jazzitup, @VinInn, @richard-cms, @yenjie, @kurtejung, @gpetruc, @istaslis, @mandrenguyen, @dgulhan, @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: 9482f86
I found an error when building:

>> Checking EDM Class Version for src/DataFormats/VertexReco/src/classes_def.xml in libDataFormatsVertexRecoCapabilities.so
>> Building shared library tmp/slc6_amd64_gcc491/src/DataFormats/HcalIsolatedTrack/src/DataFormatsHcalIsolatedTrack/libDataFormatsHcalIsolatedTrack.so
Copying tmp/slc6_amd64_gcc491/src/DataFormats/VertexReco/test/testDataFormatsVertexReco/testDataFormatsVertexReco to productstore area:
>> Building shared library tmp/slc6_amd64_gcc491/src/DataFormats/V0Candidate/src/DataFormatsV0Candidate/libDataFormatsV0Candidate.so
Copying tmp/slc6_amd64_gcc491/src/DataFormats/HcalIsolatedTrack/src/DataFormatsHcalIsolatedTrack/libDataFormatsHcalIsolatedTrack.so to productstore area:
error: class 'reco::GsfTrack' has a different checksum for ClassVersion 15. Increment ClassVersion to 16 and assign it to checksum 2463004588
Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/GsfTrackReco/src/classes_def.xml.generated with updated ClassVersion
@@@@ ----> OK  EDM Class Version 
gmake: **\* [tmp/slc6_amd64_gcc491/src/DataFormats/GsfTrackReco/src/DataFormatsGsfTrackReco/libDataFormatsGsfTrackRecoCapabilities.so] Error 1
Copying tmp/slc6_amd64_gcc491/src/DataFormats/V0Candidate/src/DataFormatsV0Candidate/libDataFormatsV0Candidate.so to productstore area:
>> Leaving Package GeneratorInterface/HiGenCommon


you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10659/7111/summary.html

[echapon]

Updating this PR too, the problem being the same as its 76X equivalent #10653

[cmsbuild]

Pull request #10659 was updated. @cmsbuild, @cvuosalo, @slava77 can you please check and sign again.

[slava77]

@cmsbuild please test

@echapon I hope you have tested that the code compiles this time
(if you think you did it last time, please share the steps; you may be doing it wrong, e.g., cms-checkdeps is missing)

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: f5021e4
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10659/7637/summary.html

[cvuosalo]

@cmsbuild please test
Comparisons stuck in Queued state

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: f5021e4
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10659/7738/summary.html

[cmsbuild]

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

[cvuosalo]

+1

For #10659 f5021e4

Small adjustments for Heavy Ion regional iterative tracking. #10653 is the 76X version of this PR, and it has already been merged.

The code changes are satisfactory, and Jenkins tests against baseline CMSSW_7_5_X_2015-08-27-2300 show a number of small differences, none significant. See #10653 for a discussion of the minor differences created by this PR and for information on processing time changes.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1