-
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
New BTV Monitor #47472
base: master
Are you sure you want to change the base?
New BTV Monitor #47472
Conversation
cms-bot internal usage |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47472/43904 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47472/43911 |
A new Pull Request was created by @aniketkhanal for master. It involves the following packages:
@antoniovagnerini, @cmsbuild, @rseidita can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Comparison SummarySummary:
|
PR description:
This PR adds a new analyzer for BTV HLT DQM. Currently, BTV uses a clone of TopMonitor, which makes it difficult to directly make changes. The new BTVMonitor uses "pat" objects instead of "reco" objects previously so all the information can be read from miniAOD level.
Instead of having a new folder for each b-tagging algorithm (which is the case currently), we now have all the b-tagging score histograms in one folder, thus reducing duplicates. The PR also adds b-tagging scores from the UParT tagger for AK4 jets, while keeping all the previously used taggers (DeepJet and ParticleNet)
PR validation:
These changes pass all the basic tests suggested in CMSSW PR instructions. Furthermore, the changes are visible in the offline DQM GUI histograms for workflow 16834.0.