-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Monitor track selection MVAs in MultiTrackValidator #16559
Conversation
Do not produce the ptcut-plots by default (i.e. also in absence of --limit-tracking-algo).
A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_8_1_X. It involves the following packages: DQMServices/ClientConfig @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
@cmsbuild , please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Generation of alternative-comparisons failed for 25202 with
(but likely not important-enough to relaunch the tests since the relmon exists) |
strange, considering that all other comparisons passed. @smuzaffar |
sorry @slava77 , you want what? |
@smuzaffar I wanted to find out if this is happening frequently. |
@slava77 , only workflow 25202.0 has this error.
I will add a check for this include some error/warning the comparison ready message. |
+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, @smuzaffar |
+1 |
Hi @makortel - could you look at fixing this problem, that seems related to this PR. Thanks ! |
Thanks David, very likely from this PR. I'll take a look. |
This PR extends MultiTrackValidator (and the plotting scripts) to optionally monitor track selection MVA variables. The motivation is to provide a monitoring tool for track selection MVA tuning.
The monitoring is enabled only for the
trackValidatorBuildingTrackingOnly
module (monitoring the "built tracks"), because the MVA variables are easily available for those, and the track collection of an iteration N is not biased by the existing track selection of itereration N. The plots includeDQMGenericClient
to be able to do the accumulation for the "cumulative histograms" also in the other directionClassifierMerger.inputClassifiers
of that iteration.In addition this PR fixes a minor bug in
makeTrackValidationPlots.py
(without any options it makes plots for the "pT>0.9 GeV" folders, while that should be active only with--ptcut
option).Tested in CMSSW_8_1_0_pre15, no changes expected in standard workflows. New histograms should appear in "trackingOnly" workflows (e.g. 1325.1, 10024.1), but there should be no changes in existing histograms.
@rovere @VinInn