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
Add plots vs. seeding layer set to MultiTrackValidator #14248
Conversation
…ple of enums and int
The seeding layer set is deduced from the seed hits. In order to have necessary information to create the histograms with fixed binning, the set of all seeding layers is given as a configuration parameter. In the python configuration side, a function to merge the seeding layer sets of all seed producers is added.
A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_8_1_X. It involves the following packages: RecoTracker/TkSeedingLayers @cvuosalo, @dmitrijus, @cmsbuild, @deguio, @slava77, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
@cmsbuild , please test |
The tests are being triggered in jenkins. |
+1
|
Could DQM (@deguio @dmitrijus @vanbesien) please review and sign? Thanks. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
+1 |
This PR adds number of tracks and fake/duplicate/pileup rates vs. seeding layer set (e.g. "BPix1+BPix2+BPix3"). The seeding layer set is deduced from the seed hits. In order to have necessary information to create the histograms with fixed binning, a list of all seeding layer sets is given as a configuration parameter. In the configuration side, the seeding layer sets of all seed producers (of a given sub-era) are merged by a new function.
Tested in 8_1_0_pre2, expecting only new histograms without changes in existing ones.
@rovere @VinInn