remove MuonHOAcceptance

[slava77]

the code was not used since the initial commit in June 2010. It can not really be used as is without major rewrite to remove use of statics and perhaps making it in a service.

[cmsbuild]

The code-checks are being triggered in jenkins.

[slava77]

@cmsbuild please test

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-21823/2830

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/25363/console Started: 2018/01/10 22:41

[cmsbuild]

A new Pull Request was created by @slava77 (Slava Krutelyov) for master.

It involves the following packages:

RecoMuon/MuonIdentification

@perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@bellan, @folguera, @abbiendi, @jhgoh, @echapon, @calderona, @HuguesBrun, @drkovalskyi, @battibass, @trocino, @bachtis, @rociovilar this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: f686d9c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-21823/25363/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
30409b0
3346696
efd30a1
52b0706
fab869c
6751577
6ea279e
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-21823/25363/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-21823/25363/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

@slava77 comparisons for the following workflows were not done due to missing matrix map:

• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-21823/1325.7_TTbar_13_94XNanoAODINPUT+TTbar_13_94XNanoAODINPUT+NANOEDMMC2017

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 26
• DQMHistoTests: Total histograms compared: 2775243
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2775073
• DQMHistoTests: Total skipped: 169
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0 KiB( 0 files compared)
• Checked 110 log files, 9 edm output root files, 26 DQM output files

[slava77]

@folguera, @drkovalskyi
please confirm that this code is not needed in some imminent development.
If we need to go back to it, there is always a github history to bring it back.
(As I mentioned in the description, as is it can not run in standard production code and needs to be rewritten anyways).

[folguera]

Hi @slava77, we have both check this PR and indeed this is not used and it is certainly not needed in some imminent development. Thanks.

[slava77]

+1

for #21823 f686d9c

• removal is OK for muon POG
• jenkins tests passed

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[fabiocos]

+1