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 SV variables to BTV OfflineDQM #36884
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36884/28147
|
A new Pull Request was created by @marco-link for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @clacaputo, @slava77, @jpata, @pmandrik, @pbo0, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hi @marco-link It was agreed by then that, the old taggers SMT, SET, etc. could be removed from validation code for clean-up in code and memory. Could this be performed now at some point? |
Hi @jfernan2, This PR is a follow up to the cleanup in which I had to remove the SV variables because they were tied to an old tagger and were using a deprecated class. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c3ce97/22222/summary.html Comparison SummarySummary:
|
@marco-link apart from the 13 new MEs added, there are many MEs which change their content with this PR. I don't have the expertise to judge if the differences are significant and in the right direction, could you please have a look? |
@jfernan2 The differences except for the jet multiplicity are as expected. The new MEs look fine, too. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36884/28197
|
Pull request #36884 was updated. @emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @clacaputo, @slava77, @jpata, @pmandrik, @pbo0, @rvenditti can you please check and sign again. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36884/28758
|
Pull request #36884 was updated. @emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @pbo0, @rvenditti can you please check and sign again. |
@jpata Can you please start testing the latest changes?? We would like to have it integrated ASAP. |
@soureek there are only DQM changes now, therefore I have not seen it in the reco list in the last weeks, and I have not commented on it. @cms-sw/dqm-l2 can offer you a review and eventually sign. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c3ce97/23178/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Addition of SV variables presented in the Btag WG Meeting.
PR validation:
runTheMatrix.py -l 12034.0
, used for validation plots to verify the fix of discriminator distributions.runTheMatrix.py -l limited -i all