New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MuonSegmentMatcher: add RPC hits, remove MuonServiceProxy dependence #10482
Conversation
A new Pull Request was created by @ptraczyk (Piotr Traczyk) for CMSSW_7_6_X. MuonSegmentMatcher: add RPC hits, remove MuonServiceProxy dependence It involves the following packages: DQMOffline/JetMET @perrotta, @cmsbuild, @cvuosalo, @fwyzard, @Martin-Grunewald, @deguio, @slava77, @danduggan can you please review it and eventually sign? Thanks. |
-1 HLT configurations must be modified in ConfDB, and only after that dumped in the release. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
DQM signature is needed here |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
+1 |
MuonSegmentMatcher: add RPC hits, remove MuonServiceProxy dependence
Added a method to MuonSegmentMatcher that performs matching of RPC hits. This is currently not used in standard RECO so no difference in reco output. Also removed the dependence on MuonServiceProxy. This resulted in a bunch of configuration files needing an update, which is also a part of this PR.