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
Clean up cosmic mu reco in pp #7541
Conversation
…isions, only keep standalone and high-levelobject
…isions, only keep standalone and high-levelobject
…isions, only keep standalone and high-levelobject
…isions, only keep standalone and high-levelobject
A new Pull Request was created by @trocino (Daniele Trocino) for CMSSW_7_4_X. Clean up cosmic mu reco in pp It involves the following packages: Configuration/StandardSequences @nclopezo, @cvuosalo, @cmsbuild, @franzoni, @slava77, @davidlange6 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
.. I'm checking it |
+1 for #7541 f376e26 In general, changes are as expected. The banchmark sample for run2 TTbar with PU35@25ns shows reduction in used resources:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged. |
This is to skip regional-tracker reco and global-muon reco (both 1- and 2-leg) in the cosmic-muon reconstruction sequence used in collisions. (Note that there is no change in cosmic runs.)
The following modules are removed:
Tracking:
regionalCosmicTracksSeq = regionalCosmicTrackerSeedingLayers * regionalCosmicTrackerSeeds * regionalCosmicCkfTrackCandidates * regionalCosmicTracks
Global-muon reco:
globalCosmicMuons, globalCosmicMuons1Leg
The corresponding event content is also removed.
This is based on the configuration prepared by @bellan. The tracking changes were discussed and agreed on with @VinInn and @rovere.